[CALCITE-6462] VolcanoPlanner internal valid may throw exception when log trace is enabled

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.37.0
• Fix Version/s: 1.38.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: xiong duan
• Created: 10/Jul/24 00:47
• Updated: 17/Oct/24 22:41
• Resolved: 17/Oct/24 22:41

Description

Unit test in JdbcAdapterTest:

@Test void testAioob_5() {
CalciteAssert.model(JdbcTest.SCOTT_MODEL)
.query("select *\n"
+ "from scott.emp e left join scott.dept d\n"
+ "on 'job' in (select job from scott.bonus b)")
.runs();
}

Exceptions:

RelSubset [rel#108:RelSubset#7.ENUMERABLE.[]] has wrong best cost {1254.5 rows, 340.5 cpu, 0.0 io}. Correct cost is {1752.0 rows, 337.5 cpu, 0.0 io}

The best RelNode and the bestCode is different.

When log  trace is enabled, Tte code will be run:

if (LOGGER.isDebugEnabled()) {
assert isValid(Litmus.THROW);
}

Issue Links

• links to: GitHub Pull Request #3851
• links to: GitHub Pull Request #3860