[SERVER-57390] Remove the implementation of the getLastError command Created: 03/Jun/21 Updated: 29/Oct/23 Resolved: 22/Jul/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 5.1.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | David Storch | Assignee: | Yoon Soo Kim |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Backwards Compatibility: | Minor Change | ||||||||||||||||||||
| Sprint: | Query Execution 2021-07-26 | ||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Case: | (copied to CRM) | ||||||||||||||||||||
| Description |
|
The command should unconditionally uassert(). Perhaps it should raise an error with code 115, CommandNotSupported? We should make sure that we continue to count attempted uses of the getLastError command in db.serverStatus().metrics.commands.getLastError, and continue to log warning 5578800. The warning looks something like this:
|
| Comments |
| Comment by Vivian Ge (Inactive) [ 06/Oct/21 ] |
|
Updating the fixversion since branching activities occurred yesterday. This ticket will be in rc0 when it’s been triggered. For more active release information, please keep an eye on #server-release. Thank you! |
| Comment by Yoon Soo Kim [ 07/Jul/21 ] |
|
batch_downconvert.h/cpp can be deleted. |