[SERVER-31003] Add clone method to Expression Created: 08/Sep/17 Updated: 06/Dec/22 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | Aggregation Framework, Querying |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | James Wahlin | Assignee: | Backlog - Query Optimization |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | neweng, pm738 | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Assigned Teams: |
Query Optimization
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Sprint: | Query 2020-03-23, Query 2020-04-06 | ||||||||
| Participants: | |||||||||
| Description |
|
Adding clone() to the Expression class in pipeline/expression.h will bring closer in line with MatchExpression. It will also allow us to avoid having to serialize / reparse in order to clone for during ExprMatchExpression::shallowClone(). |
| Comments |
| Comment by Ian Boros [ 31/Oct/19 ] |
|
Moving out of the projection epic and flagging for re-scheduling per discussion with Dave. |
| Comment by Ian Boros [ 09/Aug/19 ] |
|
I think this would make more sense a public method on Expression. Maybe we should talk in person. |
| Comment by David Storch [ 09/Aug/19 ] |
|
ian.boros, can this be built on top of the Expression walker? |