Skip to content

Commit 45d1cb5

Browse files
committed
Do not provide custom FS in config (1.5)
This is a backport of the PR duckdb#624 to `v1.5-variegata` stable branch. Workaround with providing custom FS instance as part of initial DB config is no longer necessary in recent versions. In the past it was necessary to use `allowed_directories` option. This PR removes the custom FS. Testing: the change is covered by existing test. Ref: duckdb#622
1 parent 03e5bc9 commit 45d1cb5

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

src/jni/config.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,6 @@ static duckdb::Value jobj_to_value(JNIEnv *env, const std::string &key, jobject
8181

8282
std::unique_ptr<duckdb::DBConfig> create_db_config(JNIEnv *env, jboolean read_only, jobject java_config) {
8383
auto config = std::unique_ptr<duckdb::DBConfig>(new duckdb::DBConfig());
84-
// Required for setting like 'allowed_directories' that use
85-
// file separator when checking the property value.
86-
config->file_system = duckdb::make_uniq<duckdb::VirtualFileSystem>();
8784
config->SetOptionByName("duckdb_api", "java");
8885
config->AddExtensionOption(
8986
"jdbc_stream_results",

0 commit comments

Comments
 (0)