[SERVER-62507] Investigate “TODO” comment in PathLowering::optimize() Created: 11/Jan/22 Updated: 29/Oct/23 Resolved: 27/Jan/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.3.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Drew Paroski | Assignee: | Ted Tuckman |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | auto-reverted | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Query Execution
|
||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||
| Sprint: | QO 2023-02-06 | ||||||||||||
| Participants: | |||||||||||||
| Linked BF Score: | 162 | ||||||||||||
| Description |
|
There is a “TODO” comment in PathLowering::optimize() that says "Investigate why we crash when this is removed. It should not be needed here."
The goal of this task is to determine who the author of this "TODO" comment is, follow up with them to get context, and then investigate and understand why removing the code referred to by the "TODO" comment causes these tests to fail. From there, next steps will be determined to see if the "TODO" comment should be removed and to see if any code changes are needed. |
| Comments |
| Comment by Githook User [ 27/Jan/23 ] |
|
Author: {'name': 'Ted Tuckman', 'email': 'ted.tuckman@mongodb.com', 'username': 'TedTuckman'}Message: |
| Comment by xgen-buildbaron-user [ 26/Jan/23 ] |
|
Ticket re-opened due to revert. aggregation_column_store_index_passthrough began a consistent failure of jstests/aggregation/sources/project/remove_redundant_projects.js |
| Comment by Githook User [ 26/Jan/23 ] |
|
Author: {'name': 'auto-revert-processor', 'email': 'dev-prod-dag@mongodb.com', 'username': ''}Message: Revert " This reverts commit 30d4f8bf574ebf22d3e61719c3335e075d16593c. |
| Comment by Githook User [ 25/Jan/23 ] |
|
Author: {'name': 'Ted Tuckman', 'email': 'ted.tuckman@mongodb.com', 'username': 'TedTuckman'}Message: |