[DRIVERS-219] Change showDiskLoc to showRecordId Created: 09/Apr/15 Updated: 15/Apr/19 Resolved: 21/Jun/16 |
|
| Status: | Closed |
| Project: | Drivers |
| Component/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | David Storch | Assignee: | Barrie Segal |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||||||
| Description |
|
This ticket is motivated by server changes made under The aforementioned commit deprecated the shell's showDiskLoc() helper and added a new showRecordId() helper. Drivers which offer a similar helper should consider doing the same. The $showDiskLoc query flag is still the correct flag for an OP_QUERY style find, so this change shouldn't break drivers. Note, however, that the find command field has been renamed to showRecordId. Changes made for
to something like
Drivers tests for this feature may have to be updated. |
| Comments |
| Comment by Andrew Morrow (Inactive) [ 19/Sep/15 ] |
|
Validating for C++ and C++11: Neither C++ driver has a helper related to showDiskLoc. IIUC, changes required as part of the find command will be in a separate ticket, not this one. Is that correct? |
| Comment by Bernie Hackett [ 01/Sep/15 ] |
|
Helpers aren't the issue here. If a user passes $showDiskLoc in the CRUD find() modifiers document do we convert that to showRecordId for the find command, raise an error, or..? |
| Comment by Jeffrey Yemin [ 27/Aug/15 ] |
|
Java only has a helper for this in the deprecated DBCursor API, which we don't plan on updating. |
| Comment by Bernie Hackett [ 09/Apr/15 ] |
|
jeff.yemin, craiggwilson, does this affect Java and/or .NET? |
| Comment by Bernie Hackett [ 09/Apr/15 ] |
|
I'm guessing this only affects drivers that have builders for find()/findOne(). |