MartijnVisser commented on a change in pull request #17626:
URL: https://github.com/apache/flink/pull/17626#discussion_r741022824
##########
File path: flink-table/flink-table-planner/pom.xml
##########
@@ -356,6 +356,7 @@ under the License.
<exclude>META-INF/*.DSA</exclude>
<exclude>META-INF/*.RSA</exclude>
<exclude>META-INF/services/java.sql.Driver</exclude>
+
<exclude>META-INF/versions/11/module-info.class</exclude>
Review comment:
The new Jackson is available as a multi-release JAR, which is why
there's a separate class for Java 11. The Flink shaded tests
`test_table_shaded_dependencies.sh` checks if any .*class files exist to get a
clean uber JAR. That's why I'm excluding it when creating the uber JAR.
--
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]
