[SERVER-5977] switch from list<>.sort() to sort(vector<>) Created: 31/May/12 Updated: 11/Jul/16 Resolved: 31/May/12 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Aggregation Framework |
| Affects Version/s: | 2.1.1 |
| Fix Version/s: | 2.1.2 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Chris Westin | Assignee: | Chris Westin |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Participants: |
| 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. |
| Comments |
| Comment by auto [ 31/May/12 ] |
|
Author: {u'login': u'cwestin', u'name': u'U-tellus\\cwestin', u'email': u'cwestin@10gen.com'}Message: |
| Comment by auto [ 31/May/12 ] |
|
Author: {u'login': u'cwestin', u'name': u'U-tellus\\cwestin', u'email': u'cwestin@10gen.com'}Message: |