[STORM-3966] Allow reflection in JDK11 test for storm-sql-hdfs; update guava version (apache#3575)

* [STORM-3966] Allow reflection in JDK11 test for storm-sql-hdfs; update guava version

* Fix license

---------

Co-authored-by: Richard Zowalla <[email protected]>

Author: bipinprasad

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)

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>