Consider making bounds direction checking code debug only

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Major - P3
    • 8.0.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • Query Optimization
    • Fully Compatible
    • QO 2024-02-19, QO 2024-03-04
    • 35
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      In the LargeIndexedIn workload, 15% of total CPU time is spent in isValidFor(), which validates that the index bounds are in the correct order relative to the scan and index direction (for example, the bounds [1,3] is valid for ascending index + forward traversal and descending index + backwards traversal). isValidFor() is invoked immediately after aligning the bounds to satisfy the proper direction, and only used to log and tassert if it fails.

      Given the cost in a high-value workload, we should consider disabling this check in release builds.

            Assignee:
            Ben Shteinfeld
            Reporter:
            Ben Shteinfeld
            Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              Resolved: