Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-5977

switch from list<>.sort() to sort(vector<>)

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major - P3
    • Resolution: Fixed
    • 2.1.1
    • 2.1.2
    • Aggregation Framework
    • None

    Description

      When I did the original implementation of DocumentSourceSort, there was some issue with using a vector<> of documents. Mathias spotted this in his initial scan of the code. So I went back to see what the problem was. Whatever it was, it's gone, and using a vector<> works now. This is clearly better (because it can internally do something like a qsort()), so I made the change.

      Work is already done and tested (was trivial), so I marked it for 2.1.2; will get it through code review.

      Attachments

        Activity

          People

            cwestin Chris Westin
            cwestin Chris Westin
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: