zentol commented on a change in pull request #125:
URL:
https://github.com/apache/flink-kubernetes-operator/pull/125#discussion_r837271021
##########
File path: flink-kubernetes-operator/pom.xml
##########
@@ -61,6 +61,12 @@ under the License.
<groupId>org.apache.flink</groupId>
<artifactId>flink-clients_${scala.version}</artifactId>
<version>${flink.version}</version>
+ <exclusions>
+ <exclusion>
+ <groupId>org.apache.flink</groupId>
+ <artifactId>flink-scala_${scala.version}</artifactId>
+ </exclusion>
+ </exclusions>
Review comment:
streaming-java had a transitive dependency on flink-scala in 1.14-,
which was removed in 1.15.0. To keep the dependency graph identical we added an
explicit dependency on flink-scala.
If you don't run any application within the operator itself then you
shouldn't need it.
That said you also shouldn't need a bunch of other stuff, like the QS
client, rpc-akka-loader, asm, flink-optimizer etc etc.
--
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]