openinx commented on a change in pull request #2305:
URL: https://github.com/apache/iceberg/pull/2305#discussion_r740076138
##########
File path: flink-runtime/build.gradle
##########
@@ -42,6 +42,8 @@ project(':iceberg-flink-runtime') {
implementation(project(':iceberg-nessie')) {
exclude group: 'com.google.code.findbugs', module: 'jsr305'
}
+
+ implementation "org.apache.flink:flink-connector-base"
Review comment:
@stevenzwu After we got this PR merged:
https://github.com/apache/iceberg/pull/3364, we don't need to introduce a
common `iceberg-flink-runtime` for all different flink versions, instead we
have a different `iceberg-flink:iceberg-flink-<MAJOR.MINO>-runtime` module for
different `<MAJRO.MINOR>` flink releases so that we could build the features on
top of the latest flink API.
You may want to add the transitive dependency
`org.apache.flink:flink-connector-base` in[ this line for flink
1.12](https://github.com/apache/iceberg/blob/master/flink/v1.12/build.gradle#L154)
, and[ this line for flink
1.13.](https://github.com/apache/iceberg/blob/master/flink/v1.13/build.gradle#L154)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
