Trial metrics should use boost::optional instead of 0 to indicate metric is not tracked

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Duplicate
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • Query Execution
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      We set maxMetrics to 0 to indicate a metric is not tracked in TrialRunTracker. This is bug-prone, and has led to issues such as SERVER-79088. There's no reason to use 0 instead of boost::none, so let's change this to avoid future bugs.

            Assignee:
            [DO NOT USE] Backlog - Query Execution
            Reporter:
            Matt Boros
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: