calcite icon indicating copy to clipboard operation
calcite copied to clipboard

Published 20 hours ago • verified publisher icon apache

[CALCITE-4808] Adding assertion to RelOptUtil.getVariablesUsed

Open jamesstarr opened this issue 4 years ago • 2 comments
trafficstars

  • Adding assertion to RelOptUtil.getVariablesUsed when a variable is used out of context.
  • Changing RelBuilderTest.testFilterWithCorrelationVariables which was build an illegal relNode.

jamesstarr avatar Sep 28 '21 04:09 jamesstarr

Could you merge the commits to one and rebase the PR, @jamesstarr ?

chunweilei avatar Nov 02 '21 02:11 chunweilei

I am not sure why the test is not passing. It might be something wrong with the CI.

jamesstarr avatar Nov 02 '21 20:11 jamesstarr