In sharded_agg_helpers.cpp move invariant below response status check

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 7.2.0-rc0, 7.0.5, 6.0.13, 5.0.24, 4.4.28
    • Affects Version/s: None
    • Component/s: None
    • None
    • Fully Compatible
    • ALL
    • v7.0, v6.0, v5.0, v4.4
    • QE 2023-10-16
    • 135
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      In sharded_agg_helpers.cpp when parsing explain results from remote shards, we first have an invariant that response have host and port and only after we check response status.

      However, in some tricky network failures host and port may not be set, so we need to first check response status and after assert stuff.

              Assignee:
              Ivan Fefer
              Reporter:
              Ivan Fefer
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Created:
                Updated:
                Resolved: