Move logging to bind() function

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • Query Integration
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Currently, the $log and $debugLog test extensions test logging and debug logging inside the parse() function assuming that it's called once during the course of an extension's lifetime. However, SERVER-111605 introduces a second parse() call to DocumentSourceExtension::LiteParsedExpandable.

      We should move logging into a function that's guaranteed to be called once for non-desugar extensions - bind() or compile(). I'm inclined to move it to bind() as we already have the functionality for this, whereas compile() doesn't exist yet.

            Assignee:
            Unassigned
            Reporter:
            Lynne Wang
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: