[SERVER-15709] RecordStore::Iterator::dataFor() to not take loc as an argument Created: 17/Oct/14 Updated: 10/Dec/14 Resolved: 17/Oct/14 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Storage |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Igor Canadi | Assignee: | Unassigned |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Participants: |
| Description |
|
RecordStore::Iterator can do a better job optimizing (and caching) dataFor() if it doesn't take any arguments and returns the data for the current position. All of the current uses of dataFor() use it only to return data for the current position, anyway. Patch in the attachment. |
| Comments |
| Comment by Eliot Horowitz (Inactive) [ 17/Oct/14 ] |
|
I don't think this is the direction we want to go. |