flink icon indicating copy to clipboard operation
flink copied to clipboard

[FLINK-27375][hive] Fix exception when no order keys exists in over() in Hive Dialect

Open luoyuxia opened this issue 3 years ago • 1 comments

What is the purpose of the change

To fix the exception thrown when no order keys exists in over() in Hive dialects.

Brief change log

Rewrite over() to over(order by int(1) asc) in the phase of converting Hive AST to Calcite RelNode .

Verifying this change

Additional test query in flink-connectors/flink-connector-hive/src/test/resources/query-test/window.q

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, Kubernetes/Yarn, 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? N/A

luoyuxia avatar Mar 25 '22 12:03 luoyuxia

CI report:

  • d766621dd71006d87a9f51ccbaeda1f4d7d06e31 Azure: SUCCESS
Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

flinkbot avatar Mar 25 '22 12:03 flinkbot