Consolidate unowned and owned CollatorInterface's in ExpressionContext

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Gone away
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Aggregation Framework
    • None
    • Query
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      As far as I can tell, the distinction exists because there are certain commands which require an ExpressionContext but manage a unique_ptr to the resolved collator on their own. This is confusing especially to new engineers, since its not obvious which one is valid and when.

      As part of the review for SERVER-43844, a couple of ideas involved holding by intrusive_ptr or always cloning the collator even if its "owned" elsewhere.

            Assignee:
            Backlog - Query Team (Inactive)
            Reporter:
            Nicholas Zolnierz
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: