[jira] [Commented] (FLINK-38093) Update version of Fabric8 to 7.3.1

["Mate Czagany (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/FLINK-38093?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18004931#comment-18004931 ] 

Mate Czagany commented on FLINK-38093:
--------------------------------------

Since Fabric8 has switched to Vert.x as the default HTTP implementation instead of OkHttp, we need to make a decision here:
h3. Switch from OkHttp to Vert.x
 * According to Fabric8, it's more performant than OkHttp. It should use less resources than OkHttp, but I think it will be a very small difference for Flink. Might be useful for the Flink Kubernetes Operator though
 * Some companies have internal policies to exclude Kotlin dependencies, which OkHttp pulls (FLINK-36332)
 * Vert.x has a transitive dependency of Netty, and our current version (4.1.100.Final) is not compatible. We will need to upgrade it
 * Vert.x is licensed under both the Eclipse Public License 2.0 (EPL-2.0) and the Apache License 2.0, so it can be included without a problem

h3. Stay with OkHttp
 * OkHttp is still supported by Fabric8, and will continue to be in the future
 * We still need Vert.x as a test dependency, as `kubernetes-server-mock` uses that, so a Netty upgrade is still needed (or we need to override every Netty package version in flink-kubernetes for the test scope)

 

No matter the decision, flink-kubernetes needs to have a newer Netty versions in its `test` scope after upgrading to Fabric 7.x

> Update version of Fabric8 to 7.3.1
> ----------------------------------
>
>                 Key: FLINK-38093
>                 URL: https://issues.apache.org/jira/browse/FLINK-38093
>             Project: Flink
>          Issue Type: Improvement
>          Components: Deployment / Kubernetes
>            Reporter: Mate Czagany
>            Priority: Major
>
> Fabric8 should be updated to the latest version to get the latest improvements and bug fixes available.
> It should be investigated whether it's worth it to use Vert.x, the new default HTTP library used by Fabric8 and drop OkHttp.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)