Revise misleading VersionContext::isInitialized method

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: 8.3.0-rc0
    • Component/s: None
    • None
    • Catalog and Routing
    • CAR Team 2025-11-10
    • 🟩 Routing and Topology
    • None
    • None
    • None
    • None
    • None
    • None

      The VersionContext type has an isInitialized() method to detect if an operation has an Operation FCV or not.
      This is bound to cause confusion specially when we allow an Operation FCV to be the uninitialized FCV.

       

      Rename it to a more descriptive name, e.g. hasOperationFCV().

       

      A closely related quirk is that for the kNoVersionContext and kVersionContextIgnored_UNSAFE singletons, isInitialized() returns true. We should likely revise them to be false instead.

            Assignee:
            Joan Bruguera Micó
            Reporter:
            Joan Bruguera Micó
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: