flink icon indicating copy to clipboard operation
flink copied to clipboard

[FLINK-29386][hive] Fix fail to compile flink-connector-hive for hive3

Open luoyuxia opened this issue 3 years ago • 1 comments

What is the purpose of the change

To fix fail to compile flink-connector-hive for hive3

Brief change log

  • remove the usable of TableType.INDEX_TABLE and ErrorMsg.SHOW_CREATETABLE_INDEX since they have been removed in Hive3.
  • Port DEFAULT_SERIALIZATION_FORMAT into our own code since DEFAULT_SERIALIZATION_FORMAT exist in different class between Hive2 and Hive3.
  • Fix the test for show create table in Hive3

Verifying this change

The pr run in Hive3 with the commit. The failure isn't related to this fix.

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

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

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

luoyuxia avatar Sep 22 '22 09:09 luoyuxia

CI report:

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

flinkbot avatar Sep 22 '22 09:09 flinkbot

The hive connector test in hive3 passes. Merging...

fsk119 avatar Sep 26 '22 07:09 fsk119