Skip to content

Commit 53febab

Browse files
committed
Remove the per-module Surefire --add-opens flags
The `--add-opens` flags needed by the tests of the credential store config filter, the HBase2 sink and the Kudu sink are now provided by the Surefire `argLine` of `flume-parent` (apache/logging-flume#502), which also adds the `java.base/java.lang` opens required by the `EnvironmentVariables` rule on Windows. Assisted-By: Claude Fable 5.1 <noreply@anthropic.com>
1 parent 962c10b commit 53febab

3 files changed

Lines changed: 0 additions & 32 deletions

File tree

flume-hadoop-credential-store-config-filter/pom.xml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -55,16 +55,4 @@
5555
</dependency>
5656
</dependencies>
5757

58-
<build>
59-
<plugins>
60-
<plugin>
61-
<groupId>org.apache.maven.plugins</groupId>
62-
<artifactId>maven-surefire-plugin</artifactId>
63-
<configuration>
64-
<argLine>-Djava.net.preferIPv4Stack=true --add-opens java.base/java.util=ALL-UNNAMED</argLine>
65-
</configuration>
66-
</plugin>
67-
</plugins>
68-
</build>
69-
7058
</project>

flume-hbase2-sink/pom.xml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -125,16 +125,4 @@
125125

126126
</dependencies>
127127

128-
<build>
129-
<plugins>
130-
<plugin>
131-
<groupId>org.apache.maven.plugins</groupId>
132-
<artifactId>maven-surefire-plugin</artifactId>
133-
<configuration>
134-
<argLine>-Djava.net.preferIPv4Stack=true --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/java.lang.reflect=ALL-UNNAMED --add-opens java.base/java.util=ALL-UNNAMED</argLine>
135-
</configuration>
136-
</plugin>
137-
</plugins>
138-
</build>
139-
140128
</project>

flume-kudu-sink/pom.xml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -88,14 +88,6 @@
8888

8989
<build>
9090
<plugins>
91-
<plugin>
92-
<groupId>org.apache.maven.plugins</groupId>
93-
<artifactId>maven-surefire-plugin</artifactId>
94-
<configuration>
95-
<!-- The Kudu test utilities install a fake DNS through reflection -->
96-
<argLine>-Djava.net.preferIPv4Stack=true --add-opens java.base/java.net=ALL-UNNAMED</argLine>
97-
</configuration>
98-
</plugin>
9991
<plugin>
10092
<groupId>org.apache.avro</groupId>
10193
<artifactId>avro-maven-plugin</artifactId>

0 commit comments

Comments
 (0)