From 0fd54046d2aacd71dc69925b8a2a2854fa11e9d1 Mon Sep 17 00:00:00 2001 From: Sumit Kumar Date: Wed, 18 Jan 2023 14:15:23 +0530 Subject: [PATCH] fix: fix Snowflake JDBC driver connectivity issue (#19827) ## Description - Use `JSON` format instead of `Arrow` format. - Update JDBC connector package version. - Remove redundant dependency inclusion lines from POM file. Fixes #19784 (cherry picked from commit 2a6deacbed2bd8bae5f511fb67732ce03da7008c) --- .../appsmith-plugins/snowflakePlugin/pom.xml | 34 +------------------ .../com/external/plugins/SnowflakePlugin.java | 2 ++ 2 files changed, 3 insertions(+), 33 deletions(-) diff --git a/app/server/appsmith-plugins/snowflakePlugin/pom.xml b/app/server/appsmith-plugins/snowflakePlugin/pom.xml index 5e41557f64..f3d30b4dde 100644 --- a/app/server/appsmith-plugins/snowflakePlugin/pom.xml +++ b/app/server/appsmith-plugins/snowflakePlugin/pom.xml @@ -24,42 +24,10 @@ - - - org.springframework - spring-core - provided - - - - org.springframework - spring-web - provided - - - - org.springframework - spring-webflux - - - io.projectreactor - reactor-core - - - org.springframework - spring-core - - - org.springframework - spring-web - - - - net.snowflake snowflake-jdbc - 3.13.25 + 3.13.26 diff --git a/app/server/appsmith-plugins/snowflakePlugin/src/main/java/com/external/plugins/SnowflakePlugin.java b/app/server/appsmith-plugins/snowflakePlugin/src/main/java/com/external/plugins/SnowflakePlugin.java index d9ba6876f7..7b80cb8675 100644 --- a/app/server/appsmith-plugins/snowflakePlugin/src/main/java/com/external/plugins/SnowflakePlugin.java +++ b/app/server/appsmith-plugins/snowflakePlugin/src/main/java/com/external/plugins/SnowflakePlugin.java @@ -98,6 +98,8 @@ public class SnowflakePlugin extends BasePlugin { properties.setProperty("db", String.valueOf(datasourceConfiguration.getProperties().get(1).getValue())); properties.setProperty("schema", String.valueOf(datasourceConfiguration.getProperties().get(2).getValue())); properties.setProperty("role", String.valueOf(datasourceConfiguration.getProperties().get(3).getValue())); + /* Ref: https://github.com/appsmithorg/appsmith/issues/19784 */ + properties.setProperty("jdbc_query_result_format", "json"); return Mono .fromCallable(() -> {