[SERVER-65699] Delete the test-only "sbe" command Created: 15/Apr/22 Updated: 29/Oct/23 Resolved: 22/Apr/22 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.0.0-rc6, 6.1.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | David Storch | Assignee: | Kyle Suarez |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | quick-tech-debt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||
| Backport Requested: |
v6.0
|
||||||||||||||||||||
| Sprint: | QE 2022-05-02 | ||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Linked BF Score: | 25 | ||||||||||||||||||||
| Description |
|
The SBE command no longer works for simple queries after the changes we made in the SBE plan cache and auto-parameterization project. I'd argue that we should either continue to maintain it, or just delete it. This ticket suggests the latter approach. As far as I know, the SBE test-only command was used for testing early in the development of the SBE engine but is not used any longer. It is not well tested and so we would generally not notice if we break it. I suspect that its parser and the SBE plan debug strings have already drifted out of sync. It also imposes some maintenance cost on the Query Execution team, as it can be executed by jstestfuzz and by an integration test (sbe_cmd.js). |
| Comments |
| Comment by Githook User [ 16/May/22 ] |
|
Author: {'name': 'Kyle Suarez', 'email': 'kyle.suarez@mongodb.com', 'username': 'ksuarz'}Message: |
| Comment by Liubov Molchanova [ 14/May/22 ] |
|
Requesting Backport as the failure continues to reproduce on v6.0 (BFG-1175493, BFG-1172667, BFG-1169031) |
| Comment by Githook User [ 22/Apr/22 ] |
|
Author: {'name': 'Kyle Suarez', 'email': 'kyle.suarez@mongodb.com', 'username': 'ksuarz'}Message: |