fix destination-snowflake build for jdk17 (#8854)

edgao · web-flow · commit 5dad18b377bb · 2021-12-16T12:57:31.000-08:00
diff --git a/airbyte-integrations/connectors/destination-snowflake/build.gradle b/airbyte-integrations/connectors/destination-snowflake/build.gradle
@@ -26,7 +26,7 @@ application {
 dependencies {
     implementation 'com.google.cloud:google-cloud-storage:1.113.16'
     implementation 'com.google.auth:google-auth-library-oauth2-http:0.25.5'
-    implementation 'net.snowflake:snowflake-jdbc:3.12.14'
+    implementation 'net.snowflake:snowflake-jdbc:3.13.9'
     implementation 'org.apache.commons:commons-csv:1.4'
     implementation 'com.github.alexmojaki:s3-stream-upload:2.2.2'
 
diff --git a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle
@@ -45,6 +45,9 @@ class AirbyteIntegrationTestJavaPlugin implements Plugin<Project> {
             maxHeapSize = '3g'
 
             mustRunAfter project.test
+
+            // This is needed to make the destination-snowflake tests succeed - https://github.com/snowflakedb/snowflake-jdbc/issues/589#issuecomment-983944767
+            jvmArgs = ["--add-opens=java.base/java.nio=ALL-UNNAMED"]
         }
 
         // make sure we create the integrationTest task once in case a standard source test was already initialized

Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,9 @@ class AirbyteIntegrationTestJavaPlugin implements Plugin<Project> {`
`45`	`45`	`maxHeapSize = '3g'`
`46`	`46`
`47`	`47`	`mustRunAfter project.test`
	`48`	`+`
	`49`	`+ // This is needed to make the destination-snowflake tests succeed - https://github.com/snowflakedb/snowflake-jdbc/issues/589#issuecomment-983944767`
	`50`	`+ jvmArgs = ["--add-opens=java.base/java.nio=ALL-UNNAMED"]`
`48`	`51`	`}`
`49`	`52`
`50`	`53`	`// make sure we create the integrationTest task once in case a standard source test was already initialized`