[SERVER-3449] query.cpp count exact key match mode previously used shallowEqual() (correctly I believe), now uses equal() instead Created: 19/Jul/11 Updated: 29/Feb/12 Resolved: 19/Jan/12 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Aaron Staple | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Participants: |
| Description |
|
May be less performant this way, but haven't actually tested. If we want to leave this check as equal() instead of shallowEqual(), we should change exactKeyMatch() to include numeric values as well. |
| Comments |
| Comment by Aaron Staple [ 19/Jan/12 ] |
|
I removed the comparison entirely with this commit commit 46d49e316d37667c9a500d19b91353700ca37c95 |
| Comment by Aaron Staple [ 26/Sep/11 ] |
|
@Eliot line 287 |
| Comment by Eliot Horowitz (Inactive) [ 29/Aug/11 ] |
|
Which exact line change are you referring to? |