Skip to content

[FLINK-39565] Eliminate maven build warnings across all modules #1105

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Dennis-Mircea wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[FLINK-39565] Eliminate maven build warnings across all modules #1105

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6f1217e
[hotfix] Resolve build warnings
Dennis-Mircea Apr 28, 2026
5e71a85
Fix maven.config
Dennis-Mircea Apr 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .mvn/maven.config
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
-Dorg.slf4j.simpleLogger.log.com.networknt.schema=error
34 changes: 31 additions & 3 deletions examples/autoscaling/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ under the License.
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
<version>3.0.0</version>
<!-- Version inherited from the parent pom.xml. -->
<executions>
<!-- Run shade goal on package phase -->
<execution>
Expand All @@ -125,13 +125,41 @@ under the License.
</artifactSet>
<filters>
<filter>
<!-- Do not copy the signatures in the META-INF folder.
Otherwise, this might cause SecurityExceptions when using the JAR. -->
<artifact>*:*</artifact>
<excludes>
<!-- JPMS module descriptors: the uber-JAR is not modular. -->
<exclude>module-info.class</exclude>
<exclude>META-INF/**/module-info.class</exclude>
<!-- Signature files of signed input jars. -->
<exclude>META-INF/*.SF</exclude>
<exclude>META-INF/*.DSA</exclude>
<exclude>META-INF/*.RSA</exclude>
<!-- Per-dependency Maven build metadata. -->
<exclude>META-INF/maven/**</exclude>
<!-- Per-dependency LICENSE/NOTICE/DEPENDENCIES files. -->
<exclude>META-INF/DEPENDENCIES</exclude>
<exclude>META-INF/LICENSE</exclude>
<exclude>META-INF/LICENSE.txt</exclude>
<exclude>META-INF/license.txt</exclude>
<exclude>META-INF/NOTICE</exclude>
<exclude>META-INF/NOTICE.txt</exclude>
<exclude>META-INF/licenses/**</exclude>
<!-- Kotlin compilation metadata. -->
<exclude>META-INF/*.kotlin_module</exclude>
<!-- ProGuard rules and GraalVM native-image hints. -->
<exclude>META-INF/proguard/**</exclude>
<exclude>META-INF/native-image/**</exclude>
<!-- A fresh manifest is produced by the
ManifestResourceTransformer below. -->
<exclude>META-INF/MANIFEST.MF</exclude>
</excludes>
</filter>
<filter>
<!-- okio and okio-jvm ship the same Kotlin module descriptor;
keep okio-jvm's. -->
<artifact>com.squareup.okio:okio</artifact>
<excludes>
<exclude>META-INF/okio.kotlin_module</exclude>
</excludes>
</filter>
</filters>
Expand Down
130 changes: 126 additions & 4 deletions examples/flink-beam-example/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,22 @@ under the License.
<groupId>org.springframework</groupId>
<artifactId>spring-jcl</artifactId>
</exclusion>
<!--
Excluded because the example does not use any Beam ML
transform and beam-sdks-java-extensions-ml:2.62.0 ships
a malformed POM (missing <version> for
com.google.http-client:google-http-client and
com.google.api:gax), which causes Maven to log:
[WARNING] The POM for
org.apache.beam:beam-sdks-java-extensions-ml:jar:2.62.0
is invalid, transitive dependencies (if any) will not be available
Excluding it avoids resolving the broken POM altogether
and also slims down the resulting uber-JAR.
-->
<exclusion>
<groupId>org.apache.beam</groupId>
<artifactId>beam-sdks-java-extensions-ml</artifactId>
</exclusion>
</exclusions>
</dependency>

Expand Down Expand Up @@ -120,7 +136,7 @@ under the License.
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
<version>3.1.1</version>
<!-- Version inherited from the parent pom.xml. -->
<executions>
<!-- Run shade goal on package phase -->
<execution>
Expand All @@ -139,19 +155,125 @@ under the License.
</artifactSet>
<filters>
<filter>
<!-- Do not copy the signatures in the META-INF folder.
Otherwise, this might cause SecurityExceptions when using the JAR. -->
<artifact>*:*</artifact>
<excludes>
<exclude>META-INF/versions/9/module-info.class</exclude>
<!-- JPMS module descriptors: the uber-JAR is not modular. -->
<exclude>module-info.class</exclude>
<exclude>META-INF/**/module-info.class</exclude>
<!-- Signature files of signed input jars. -->
<exclude>META-INF/*.SF</exclude>
<exclude>META-INF/*.DSA</exclude>
<exclude>META-INF/*.RSA</exclude>
<!-- Per-dependency Maven build metadata. -->
<exclude>META-INF/maven/**</exclude>
<!-- Per-dependency LICENSE/NOTICE/DEPENDENCIES files
(multiple casings & paths used across the Beam dep tree). -->
<exclude>META-INF/DEPENDENCIES</exclude>
<exclude>META-INF/LICENSE</exclude>
<exclude>META-INF/LICENSE.txt</exclude>
<exclude>META-INF/license.txt</exclude>
<exclude>META-INF/NOTICE</exclude>
<exclude>META-INF/NOTICE.txt</exclude>
<exclude>META-INF/license/**</exclude>
<exclude>META-INF/licenses/**</exclude>
<exclude>LICENSE</exclude>
<exclude>LICENSE.txt</exclude>
<exclude>LICENSE-*.txt</exclude>
<exclude>about.html</exclude>
<!-- Kotlin compilation metadata + Kotlin Multiplatform
common-source metadata (shipped by some wire artifacts);
none is consumed at runtime by the JVM. -->
<exclude>META-INF/*.kotlin_module</exclude>
<exclude>META-INF/kotlin-project-structure-metadata.json</exclude>
<exclude>commonMain/**</exclude>
<!-- ProGuard rules and GraalVM native-image hints. -->
<exclude>META-INF/proguard/**</exclude>
<exclude>META-INF/native-image/**</exclude>
<!-- Build-time git/version metadata. -->
<exclude>arrow-git.properties</exclude>
<!-- Netty release metadata; one wins anyway. -->
<exclude>META-INF/io.netty.versions.properties</exclude>
<!-- Mozilla public-suffix list shipped by httpclient and
beam-vendor-grpc; harmless duplicate. -->
<exclude>mozilla/public-suffix-list.txt</exclude>
<!-- A fresh manifest is produced by the
ManifestResourceTransformer below. -->
<exclude>META-INF/MANIFEST.MF</exclude>
</excludes>
</filter>
<filter>
<!--
kotlinpoet (the legacy artifact) is fully superseded by
kotlinpoet-jvm in Kotlin 1.7+'s multiplatform layout, but
Beam still pulls both in transitively. Drop kotlinpoet's
classes so kotlinpoet-jvm wins deterministically.
-->
<artifact>com.squareup:kotlinpoet</artifact>
<excludes>
<exclude>**</exclude>
</excludes>
</filter>
<filter>
<!--
Same situation for wire-runtime / wire-schema: the *-jvm
artifacts are the canonical JVM ones, the no-suffix ones
are stale multiplatform metadata bundles.
-->
<artifact>com.squareup.wire:wire-runtime</artifact>
<excludes>
<exclude>**</exclude>
</excludes>
</filter>
<filter>
<artifact>com.squareup.wire:wire-schema</artifact>
<excludes>
<exclude>**</exclude>
</excludes>
</filter>
<filter>
<!--
wire-compiler ships a shaded copy of a few wire-schema
classes (DirectedAcyclicGraph, PartitionedSchema, etc.)
that overlap with wire-schema-jvm. wire-schema-jvm is the
canonical runtime artifact, so drop those duplicates from
wire-compiler.
-->
<artifact>com.squareup.wire:wire-compiler</artifact>
<excludes>
<exclude>com/squareup/wire/schema/DirectedAcyclicGraph*.class</exclude>
<exclude>com/squareup/wire/schema/PartitionedSchema*.class</exclude>
<exclude>com/squareup/wire/schema/Target*.class</exclude>
<exclude>com/squareup/wire/schema/WireRun*.class</exclude>
</excludes>
</filter>
<filter>
<!--
wire-schema-jvm ships its own copies of google.protobuf.*
.proto descriptors that overlap with protobuf-java.
protobuf-java is the canonical source, so drop them.
-->
<artifact>com.squareup.wire:wire-schema-jvm</artifact>
<excludes>
<exclude>google/protobuf/*.proto</exclude>
</excludes>
</filter>
<filter>
<!--
beam-sdks-java-harness re-bundles all the classes from
beam-runners-core-java (~280 classes). Drop them from the
harness so beam-runners-core-java's are used.
-->
<artifact>org.apache.beam:beam-sdks-java-harness</artifact>
<excludes>
<exclude>org/apache/beam/runners/core/**</exclude>
</excludes>
</filter>
</filters>
<transformers>
<transformer implementation="org.apache.maven.plugins.shade.resource.ServicesResourceTransformer"/>
<!-- Produce a fresh MANIFEST.MF for the uber-JAR; the inputs'
manifests are dropped via the filter above. -->
<transformer implementation="org.apache.maven.plugins.shade.resource.ManifestResourceTransformer"/>
</transformers>
</configuration>
</execution>
Expand Down
35 changes: 32 additions & 3 deletions examples/flink-sql-runner-example/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ under the License.
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
<version>3.1.1</version>
<!-- Version inherited from the parent pom.xml. -->
<executions>
<!-- Run shade goal on package phase -->
<execution>
Expand All @@ -113,13 +113,42 @@ under the License.
</artifactSet>
<filters>
<filter>
<!-- Do not copy the signatures in the META-INF folder.
Otherwise, this might cause SecurityExceptions when using the JAR. -->
<artifact>*:*</artifact>
<excludes>
<!-- JPMS module descriptors: the uber-JAR is not modular. -->
<exclude>module-info.class</exclude>
<exclude>META-INF/**/module-info.class</exclude>
<!-- Signature files of signed input jars: trigger
SecurityException at runtime if kept in the uber-JAR. -->
<exclude>META-INF/*.SF</exclude>
<exclude>META-INF/*.DSA</exclude>
<exclude>META-INF/*.RSA</exclude>
<!-- Per-dependency Maven build metadata. -->
<exclude>META-INF/maven/**</exclude>
<!-- Per-dependency LICENSE/NOTICE/DEPENDENCIES files. -->
<exclude>META-INF/DEPENDENCIES</exclude>
<exclude>META-INF/LICENSE</exclude>
<exclude>META-INF/LICENSE.txt</exclude>
<exclude>META-INF/license.txt</exclude>
<exclude>META-INF/NOTICE</exclude>
<exclude>META-INF/NOTICE.txt</exclude>
<exclude>META-INF/licenses/**</exclude>
<!-- Kotlin compilation metadata. -->
<exclude>META-INF/*.kotlin_module</exclude>
<!-- ProGuard rules and GraalVM native-image hints. -->
<exclude>META-INF/proguard/**</exclude>
<exclude>META-INF/native-image/**</exclude>
<!-- A fresh manifest is produced by the
ManifestResourceTransformer below. -->
<exclude>META-INF/MANIFEST.MF</exclude>
</excludes>
</filter>
<filter>
<!-- okio and okio-jvm ship the same Kotlin module descriptor;
keep okio-jvm's. -->
<artifact>com.squareup.okio:okio</artifact>
<excludes>
<exclude>META-INF/okio.kotlin_module</exclude>
</excludes>
</filter>
</filters>
Expand Down
Loading