snuyanzin commented on PR #5: URL: https://github.com/apache/flink-connector-hive/pull/5#issuecomment-1846072081
Currently it works for hive2 and hive3 (without e2e for jdk11) as |hive version | flink 1.18 + jdk 8 | flink 1.19-snapshot + jdk8 | flink 1.18 + jdk 11 | flink 1.19-snapshot + jdk11 | |----------|:-------------:|------:|------------------------------:|---------------------:| | 2 | ✅ | ✅ |✅ |✅ | |3 | ✅ |✅ | no E2E | no E2E| as mentioned above under flink main there is no E2E tests for jdk11 as well The PR currently copies `ci.yaml` from [flink-connector-shared-utils](https://github.com/apache/flink-connector-shared-utils) ... at the same side there are 2 PRs for that repo to support customization of jdk version https://github.com/apache/flink-connector-shared-utils/pull/24 and maven profiles https://github.com/apache/flink-connector-shared-utils/pull/29 , these 2 features are used here. Once they are merged `ci.yml` could be reused and the copy could be removed @luoyuxia , @MartijnVisser please let me know whether you are ok with this or not -- 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. To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
