Skip to content

Commit

Permalink
Make arrow-memory-core and arrow-memory-unsafe automatic modules
Browse files Browse the repository at this point in the history
  • Loading branch information
jduo committed Dec 11, 2023
1 parent a7d2267 commit 8931e95
Show file tree
Hide file tree
Showing 4 changed files with 26 additions and 74 deletions.
35 changes: 11 additions & 24 deletions java/memory/memory-core/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,17 @@

<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
<archive>
<manifestEntries>
<Automatic-Module-Name>org.apache.arrow.memory.core</Automatic-Module-Name>
</manifestEntries>
</archive>
</configuration>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
Expand All @@ -53,30 +64,6 @@
</build>

<profiles>
<profile>
<id>error-prone-jdk11+</id>
<activation>
<jdk>[11,]</jdk>
<property>
<name>!m2e.version</name>
</property>
</activation>
<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration combine.self="override">
<argLine>--add-opens=java.base/java.lang.reflect=org.apache.arrow.memory.core --add-opens=java.base/java.nio=org.apache.arrow.memory.core,ALL-UNNAMED</argLine>
<excludes>
<!-- Test is only useful when NOT running with add-opens -->
<exclude>**/TestOpens.java</exclude>
</excludes>
</configuration>
</plugin>
</plugins>
</build>
</profile>
<profile>
<id>opens-tests</id>
<!-- Run tests WITHOUT add-opens to make sure we fail-fast -->
Expand Down
28 changes: 0 additions & 28 deletions java/memory/memory-core/src/main/java/module-info.java

This file was deleted.

15 changes: 15 additions & 0 deletions java/memory/memory-unsafe/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -33,4 +33,19 @@
</dependency>
</dependencies>

<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
<archive>
<manifestEntries>
<Automatic-Module-Name>org.apache.arrow.memory.unsafe</Automatic-Module-Name>
</manifestEntries>
</archive>
</configuration>
</plugin>
</plugins>
</build>
</project>
22 changes: 0 additions & 22 deletions java/memory/memory-unsafe/src/main/java/module-info.java

This file was deleted.

0 comments on commit 8931e95

Please sign in to comment.