## What is the purpose of the change
This PR tried to fix insert into with order by. It was caused by calcite 
validate rewrite and change SqlNode in some cases such as ORDER_BY sql kind. In 
these special cases validate return new SqlNode with different kind, however, 
flink does not use this new SqlNode.
In addition, `DataStreamSortRule` seems require the first sort attributes is 
time attributes and order is ascending

## Brief change log
TableEnvironment validate

## Verifying this change

test case

## Does this pull request potentially affect one of the following parts:

  - Dependencies (does it add or upgrade a dependency): (no)
  - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (no)
  - The serializers: (no)
  - The runtime per-record code paths (performance sensitive): (no)
  - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
  - The S3 file system connector: (no)

## Documentation

  - Does this pull request introduce a new feature? (no)
  - If yes, how is the feature documented? (not documented)


[ Full content available at: https://github.com/apache/flink/pull/6648 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to