[SERVER-76770] Split up SBE VM implementation into multiple source files Created: 02/May/23 Updated: 30/Jan/24 |
|
| Status: | Open |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor - P4 |
| Reporter: | David Storch | Assignee: | Backlog - Query Execution |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Query Execution
|
| Participants: |
| Description |
|
The SBE's VM for expression execution is more than 8000 lines long and growing. See vm.cpp. We should consider splitting it into a few pieces. One idea for how to split it is to move all the code implementing builtin functions into a new cpp file, leaving the meat of the bytecode interpreter in place. |
| Comments |
| Comment by Kyle Suarez [ 09/Jun/23 ] |
|
Moving to quick wins. |
| Comment by Ana Meza [ 23/May/23 ] |
|
Passing to Directors to find one assignee |