[Xapian-tickets] [Xapian] #359: MultiValueSorter::add() and Enquire::set_sort_by_value() inconsistency

[Xapian]

#359: MultiValueSorter::add() and Enquire::set_sort_by_value() inconsistency
-------------------------+--------------------------------------------------
 Reporter:  olly         |       Owner:  olly    
     Type:  defect       |      Status:  assigned
 Priority:  highest      |   Milestone:  1.1.3   
Component:  Library API  |     Version:  1.1.0   
 Severity:  normal       |    Keywords:          
Blockedby:               |    Platform:  All     
 Blocking:               |  
-------------------------+--------------------------------------------------

Comment(by olly):

 Well, collapsing with a pairwise operator would require a major
 reimplementation of the current approach.  In fact, I'm not sure it can be
 done efficiently if the operator is "these two are equivalent" we need to
 compare a candidate to all current proto-mset entries.  If it's an
 operator< sort order, we have the same issues as with sorting where the
 user can crash or hang the sort implementation by violating the well-
 ordered requirement.  It's more consistent to have generated keys for both
 too.

-- 
Ticket URL: <http://trac.xapian.org/ticket/359#comment:6>
Xapian <http://xapian.org/>
Xapian