[SERVER-85767] Refactor classic_runtime_planner_for_sbe.cpp into smaller files Created: 26/Jan/24 Updated: 28/Jan/24 Resolved: 28/Jan/24 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 7.3.0-rc2 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Adi Agrawal | Assignee: | Adi Agrawal |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Sprint: | QE 2024-02-05 |
| Participants: | |
| Story Points: | 0.5 |
| Description |
|
Currently classic_runtime_planner_for_sbe.h, and classic_runtime_planner_for_sbe.cpp have methods for various different classes. It's cleaner to separate them into different files like so:
Since there is only one consumer (get_executor.cpp) for classic_runtime_planner_for_sbe.h as it stands today and it would need all the subfiles, it would be ideal to have one header file and multiple specialized cpp files. |
| Comments |
| Comment by Githook User [ 27/Jan/24 ] |
|
Author: {'name': 'Adityavardhan Agrawal', 'email': 'adi.agrawal@mongodb.com', 'username': 'Adityav369'}Message: GitOrigin-RevId: 825e386d34dd01036a92e9cf90649a776b7a88dc |