danny0405 opened a new pull request #9712: [FLINK-13656] 
[sql-parser][table-planner][table-planner-blink] Bump Calcite dependency to 
1.21.0
URL: https://github.com/apache/flink/pull/9712
 
 
   ### SQL Parser
   Some refactoring in flink-sql-parser
   
   ### Flink Planner
   - `org.apache.calcite.rel.core.Match#partitionKeys` has changed from 
`List<RexNode>` to `ImmutableBitSet`, since CALCITE-1935,
   - `org.apache.calcite.rel.core.Match#copy` has been removed since 
CALCITE-1935 part1.
   - 
`org.apache.flink.table.api.stream.sql.JoinTest#testRightOuterJoinEquiAndLocalPred`,
 `#testLeftOuterJoinEquiAndLocalPred`, 
`org.apache.flink.table.api.batch.sql.JoinTest#testFullOuterJoinEquiAndLocalPred`,
 `#testRightOuterJoinEquiAndLocalPred`, `#testLeftOuterJoinEquiAndLocalPred` 
plan has changed, but it is still correct. since CALCITE-3101
   - ConverterRule that use `RelOptRule#description` was affected because the
   default description generation rule has changed since CALCITE-3115.
   - 
`org.apache.flink.table.api.stream.table.AggregateTest#testMultiDistinctAggregateSameFieldOnHopWindow`,
 
`org.apache.flink.table.api.stream.sql.DistinctAggregateTest#testMultiDistinctAggregateSameFieldOnHopWindow`
 max distinct was removed. Since CALCITE-3159
   
   ### Blink Planner
   
   - The plan change mostly because of the same reason with flink planner
   - `SqlFunction` was copied from CALCITE because of CALCITE-3360
   - org.apache.flink.table.planner.plan.batch.sql#testMinusWithNestedTypes IS 
NOT DISTINCT FROM is not expanded now since CALCITE-3174
   - `FlinkTypeFactory#createDecimalQuotient` was moved to 
`FlinkTypeSystem#deriveDecimalDivideType `due to CALCITE-3187
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to