Investigate calling serverGlobalParams.featureCompatibility.isVersionInitialized() in isEnabled()

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Major - P3
    • 7.1.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Replication
    • Fully Compatible
    • Repl 2023-05-01
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Currently FeatureFlag::isEnabled() does not check if serverGlobalParams.featureCompatibility.isVersionInitialized(), but in many places throughout the codebase the caller needs to check this (i.e. serverGlobalParams.featureCompatibility.isVersionInitialized() &&
      gFeatureFlagRequireTenantID.isEnabled(serverGlobalParams.featureCompatibility))

      We should investigate if we should always be calling this and can move it into isEnabled()

      Edit: SERVER-66482 created isEnabledUseDefaultFCVWhenUninitialized so we should audit usages of isEnabled and see if we can use isEnabledUseDefaultFCVWhenUninitialized instead.

              Assignee:
              Jiawei Yang
              Reporter:
              Huayu Ouyang
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: