Skip to content

2.27.0.0-b369

@mtakahar mtakahar tagged this 24 Jul 18:44
Summary:
Disable Memoize under BNL because it couldn't find any cache hit at all when it is hooked up under BNL.   This was not only adding slight CPU overhead but reducing the costs of BNL by non-existing benefit.

Proper BNL + Memoize support would require change to Memoize to deal with the batched input.

See https://github.com/yugabyte/yugabyte-db/issues/25251 for examples of the issue and potential change to Memoize.
Jira: DB-14443

Test Plan:
./yb_build.sh release --java-test 'org.yb.pgsql.TestPgRegressJoin'
./yb_build.sh release --java-test 'org.yb.pgsql.TestPgRegressExtension'
./yb_build.sh release --java-test 'org.yb.pgsql.TestPgRegressThirdPartyExtensionsPgHintPlan'
./yb_build.sh release --java-test 'org.yb.pgsql.TestPgRegressMisc'
./yb_build.sh release --java-test 'org.yb.pgsql.TestPgRegressPlanner'

Reviewers: gkukreja, william.mckenna

Reviewed By: william.mckenna

Subscribers: mihnea, smishra, yql

Differential Revision: https://phorge.dev.yugabyte.com/D45485
Assets 2
Loading