This is an automated email from the ASF dual-hosted git repository.

rzo1 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/storm.git


The following commit(s) were added to refs/heads/master by this push:
     new 7194ac4e1 [STORM-3966] Allow reflection in JDK11 test for 
storm-sql-hdfs; update guava version (#3575)
7194ac4e1 is described below

commit 7194ac4e1d832f3246dab2fb163e4f59a8910268
Author: Bipin Prasad <[email protected]>
AuthorDate: Mon Oct 23 10:35:44 2023 -0700

    [STORM-3966] Allow reflection in JDK11 test for storm-sql-hdfs; update 
guava version (#3575)
    
    * [STORM-3966] Allow reflection in JDK11 test for storm-sql-hdfs; update 
guava version
    
    * Fix license
    
    ---------
    
    Co-authored-by: Richard Zowalla <[email protected]>
---
 DEPENDENCY-LICENSES                           |  1 +
 sql/storm-sql-external/storm-sql-hdfs/pom.xml | 31 ++++++++++++++++++++++++---
 2 files changed, 29 insertions(+), 3 deletions(-)

diff --git a/DEPENDENCY-LICENSES b/DEPENDENCY-LICENSES
index b8b4df5e8..d6571e7c4 100644
--- a/DEPENDENCY-LICENSES
+++ b/DEPENDENCY-LICENSES
@@ -197,6 +197,7 @@ List of third-party dependencies grouped by their license 
type.
         * Graphite Integration for Metrics 
(io.dropwizard.metrics:metrics-graphite:3.2.6 - 
http://metrics.dropwizard.io/metrics-graphite/)
         * Gson (com.google.code.gson:gson:2.2.4 - 
http://code.google.com/p/google-gson/)
         * Guava: Google Core Libraries for Java (com.google.guava:guava:16.0.1 
- http://code.google.com/p/guava-libraries/guava)
+        * Guava: Google Core Libraries for Java (com.google.guava:guava:19.0 - 
https://github.com/google/guava/guava)
         * Guava: Google Core Libraries for Java 
(com.google.guava:guava:27.0.1-jre - https://github.com/google/guava/guava)
         * Guava InternalFutureFailureAccess and InternalFutures 
(com.google.guava:failureaccess:1.0.1 - 
https://github.com/google/guava/failureaccess)
         * Guava ListenableFuture only 
(com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava - 
https://github.com/google/guava/listenablefuture)
diff --git a/sql/storm-sql-external/storm-sql-hdfs/pom.xml 
b/sql/storm-sql-external/storm-sql-hdfs/pom.xml
index 4207a1fb9..7f1dbbec9 100644
--- a/sql/storm-sql-external/storm-sql-hdfs/pom.xml
+++ b/sql/storm-sql-external/storm-sql-hdfs/pom.xml
@@ -28,8 +28,18 @@
     <artifactId>storm-sql-hdfs</artifactId>
     
     <properties>
-        <!-- Required downgrade by calcite-core 1.14.0 -->
-        <guava.version>16.0.1</guava.version>
+        <!--
+             +- org.apache.storm:storm-hdfs:jar:2.6.0-SNAPSHOT:provided
+             |  +- org.apache.hadoop:hadoop-auth:jar:2.10.1:provided
+
+             to fix Illegal reflective access by 
org.apache.hadoop.security.authentication.util.KerberosUtil
+             (org.apache.hadoop:hadoop-auth:2.10.1)
+             to method sun.security.krb5.Config.getInstance() add
+             "add-opens java.security.jgss/sun.security.krb5=ALL-UNNAMED"
+        -->
+        <storm-sql-hdfs.test.introspection.argLine>--add-opens 
java.security.jgss/sun.security.krb5=ALL-UNNAMED</storm-sql-hdfs.test.introspection.argLine>
+        <!-- Required downgrade by calcite-core 1.16.0 
https://mvnrepository.com/artifact/org.apache.calcite/calcite-core/1.16.0 -->
+        <guava.version.downgrade>19.0</guava.version.downgrade>
     </properties>
 
     <developers>
@@ -109,7 +119,7 @@
         <dependency>
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>
-            <version>${guava.version}</version>
+            <version>${guava.version.downgrade}</version>
         </dependency>
     </dependencies>
     <build>
@@ -130,6 +140,21 @@
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-pmd-plugin</artifactId>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <configuration>
+                    <argLine>${storm-sql-hdfs.test.introspection.argLine} 
-Xmx3g -XX:+HeapDumpOnOutOfMemoryError</argLine>
+                    <redirectTestOutputToFile>true</redirectTestOutputToFile>
+                    <excludedGroups>IntegrationTest | 
${java.unit.test.exclude.groups}</excludedGroups>
+                    <trimStackTrace>false</trimStackTrace>
+                    <forkCount>1.0C</forkCount>
+                    <reuseForks>true</reuseForks>
+                    <systemPropertyVariables>
+                        
<storm.home>${project.basedir}/target/testhome</storm.home>
+                    </systemPropertyVariables>
+                </configuration>
+            </plugin>
         </plugins>
     </build>
 </project>

Reply via email to