Pass IndexBounds directly to the CheckBounds stage

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 6.0.0-rc2, 6.1.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Fully Compatible
    • ALL
    • v6.0
    • QE 2022-05-02
    • 135
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      As part of SERVER-63351 we passed IndexBounds to the CheckBounds stage as an EExpression, either as a Variable with the Slot in the Runtime Environment, or as a Constant.

      Upon evaluation both of them are not owned, resulting in performing unnecessary copy to the CheckBounds stage, causing the regression BF-24874.

      As a solution to this issue, we will not pass the IndexBounds as an EExpression, but rather as a SlotId or as IndexBounds*.

      This should solve the performance issue and make it even more explicit of how to instantiate the CheckBounds stage.

            Assignee:
            Denis Grebennicov
            Reporter:
            Denis Grebennicov
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: