Ignore path option passed by Spark

Apache Spark passes the `path` option when a SELECT on a JDBC DataSource
table is performed. It is the internal Spark option and is likely
passed by mistake, so we need to ignore it to allow the connection
to be established.

Testing: new test added that checks that the `path` option is ognored.

Fixes: duckdb#55

Author: staticlibs

CMakeLists.txt

@@ -561,6 +561,7 @@ add_library(duckdb_java SHARED
   src/jni/duckdb_java.cpp
   src/jni/functions.cpp
   src/jni/refs.cpp
+  src/jni/types.cpp
   src/jni/util.cpp
   ${DUCKDB_SRC_FILES})
 

CMakeLists.txt.in

@@ -103,6 +103,7 @@ add_library(duckdb_java SHARED
   src/jni/duckdb_java.cpp
   src/jni/functions.cpp
   src/jni/refs.cpp
+  src/jni/types.cpp
   src/jni/util.cpp
   ${DUCKDB_SRC_FILES})
 

src/main/java/org/duckdb/DuckDBDriver.java

@@ -38,6 +38,13 @@ public Connection connect(String url, Properties info) throws SQLException {
             read_only = prop_clean.equals("1") || prop_clean.equals("true") || prop_clean.equals("yes");
         }
         info.put("duckdb_api", "jdbc");
+
+        // Apache Spark passes this option when SELECT on a JDBC DataSource
+        // table is performed. It is the internal Spark option and is likely
+        // passed by mistake, so we need to ignore it to allow the connection
+        // to be established.
+        info.remove("path");
+
         return DuckDBConnection.newConnection(url, read_only, info);
     }
 

src/test/java/org/duckdb/TestDuckDBJDBC.java

@@ -4793,6 +4793,13 @@ public static void test_client_config_retained_on_dup() throws Exception {
         }
     }
 
+    public static void test_spark_path_option_ignored() throws Exception {
+        Properties config = new Properties();
+        config.put("path", "path/to/spark/catalog/dir");
+        Connection conn = DriverManager.getConnection(JDBC_URL, config);
+        conn.close();
+    }
+
     public static void main(String[] args) throws Exception {
         String arg1 = args.length > 0 ? args[0] : "";
         final int statusCode;