Uploaded image for project: 'Terrier Core'
  1. Terrier Core
  2. TR-251

ResultSet implementation should know how to sort themselves

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.6
    • Component/s: None
    • Labels:
      None

      Description

      We now have various implementations of ResultSet. These various implementations should know how to sort themselves (if possible), instead of various code spread around for sorting them.

        Attachments

          Activity

          Hide
          craigm Craig Macdonald added a comment -

          Tagging for 3.6

          Show
          craigm Craig Macdonald added a comment - Tagging for 3.6
          Hide
          craigm Craig Macdonald added a comment -

          Committed revision 3561.

          Show
          craigm Craig Macdonald added a comment - Committed revision 3561.
          Hide
          craigm Craig Macdonald added a comment -

          I have implemented an initial version of this. The ResultSet interface is modified as follows:

          /** Sorts all documents in this resultset by descending score */
          void sort();
          
          /** Sorts the top <tt>topDocs</tt> document in this resultset be first.
           * The order of the remaining documents is undefined.
           * @param topDocs number of documents to top-rank
           */
          void sort(int topDocs);
          
          Show
          craigm Craig Macdonald added a comment - I have implemented an initial version of this. The ResultSet interface is modified as follows: /** Sorts all documents in this resultset by descending score */ void sort(); /** Sorts the top <tt>topDocs</tt> document in this resultset be first. * The order of the remaining documents is undefined. * @param topDocs number of documents to top-rank */ void sort( int topDocs);
          Hide
          craigm Craig Macdonald added a comment -

          Tagging for 4.0. Major API change.

          Show
          craigm Craig Macdonald added a comment - Tagging for 4.0. Major API change.

            People

            • Assignee:
              craigm Craig Macdonald
              Reporter:
              craigm Craig Macdonald
            • Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: