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

      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: