Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-49876

Fix always-false if statement in CatalogCache

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Gone away
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Sharding
    • Operating System:
      ALL
    • Sprint:
      Sharding 2020-11-16, Sharding 2020-11-30, Sharding 2020-12-14, Sharding 2020-12-28, Sharding 2021-01-11, Sharding 2021-01-25, Sharding 2021-02-22, Sharding 2021-03-08, Sharding 2021-03-22, Sharding 2021-04-05, Sharding 2021-04-19, Sharding 2021-05-03, Sharding EMEA 2021-11-15, Sharding EMEA 2021-11-29

      Description

      We check if the existingRoutingInfo exists when attempting to reset stale shards. It will never exist, because we've std::moved the shared pointer into refreshCollectionRoutingInfo.

      We should figure out a way to achieve the desired behavior while maintaining the std::move semantics.

        Attachments

          Issue Links

          related to

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-49800 add use-after-move rule to clang-tidy evergreen builder

          • Major - P3 - Major loss of function.
          • Closed

            Activity

              People

              Assignee:
              kaloian.manassiev Kaloian Manassiev
              Reporter:
              blake.oler Blake Oler
              Participants:
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: