Skip query knob configuration materialization on IDHACK caching fast path

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Major - P3
    • 9.1.0-rc0, 9.0.0-rc1
    • Affects Version/s: None
    • Component/s: None
    • None
    • Query Execution
    • Fully Compatible
    • v9.0
    • 200
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      BF-44503: instruction-count regression on BM_FIND_ONE (benchmarks_sep) from 8888ec0.

      CanonicalQuery::initCq evaluated getQueryKnobConfiguration().getDisablePlanCache() first when computing _disablePlanCache. That call materializes the knob configuration, adding work to the IDHACK fast path even though IDHACK queries are never cached.

      Reorder so isIdHackQuery() short-circuits first, avoiding materialization for IDHACK and restoring the BM_FIND_ONE instruction count.

            Assignee:
            Catalin Sumanaru
            Reporter:
            Catalin Sumanaru
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: