Skip to content

Commit d72b7a4

Browse files
committed
Actually untangle patches
1 parent 5457db5 commit d72b7a4

File tree

2 files changed

+27
-69
lines changed

2 files changed

+27
-69
lines changed

patches/duckdb/extension_install_rework.patch

Lines changed: 27 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
diff --git a/src/include/duckdb/main/database.hpp b/src/include/duckdb/main/database.hpp
2-
index ed956daa64..d4774faac5 100644
2+
index d3c5fb9bd5..b3d0aaa09e 100644
33
--- a/src/include/duckdb/main/database.hpp
44
+++ b/src/include/duckdb/main/database.hpp
5-
@@ -97,6 +97,10 @@ private:
6-
shared_ptr<LogManager> log_manager;
5+
@@ -100,6 +100,10 @@ private:
6+
unique_ptr<ExternalFileCache> external_file_cache;
77

88
duckdb_ext_api_v1 (*create_api_v1)();
99
+public:
@@ -31,10 +31,10 @@ index 6ccd1a1156..8040f537b6 100644
3131
//! Debugging repositories (target local, relative paths that are produced by DuckDB's build system)
3232
static constexpr const char *BUILD_DEBUG_REPOSITORY_PATH = "./build/debug/repository";
3333
diff --git a/src/main/database.cpp b/src/main/database.cpp
34-
index 084dab6f30..6e21356bd9 100644
34+
index db6e1ed445..d495aab058 100644
3535
--- a/src/main/database.cpp
3636
+++ b/src/main/database.cpp
37-
@@ -344,6 +344,28 @@ DuckDB::DuckDB(DatabaseInstance &instance_p) : instance(instance_p.shared_from_t
37+
@@ -356,6 +356,28 @@ DuckDB::DuckDB(DatabaseInstance &instance_p) : instance(instance_p.shared_from_t
3838
DuckDB::~DuckDB() {
3939
}
4040

@@ -63,19 +63,30 @@ index 084dab6f30..6e21356bd9 100644
6363
SecretManager &DatabaseInstance::GetSecretManager() {
6464
return *config.secret_manager;
6565
}
66+
@@ -507,6 +529,10 @@ idx_t DuckDB::NumberOfThreads() {
67+
68+
bool DatabaseInstance::ExtensionIsLoaded(const std::string &name) {
69+
auto extension_name = ExtensionHelper::GetExtensionName(name);
70+
+ if (extension_name == "httpfs") {
71+
+ ExtensionInstallInfo info;
72+
+ SetExtensionLoaded(extension_name, info);
73+
+ }
74+
auto it = loaded_extensions_info.find(extension_name);
75+
return it != loaded_extensions_info.end() && it->second.is_loaded;
76+
}
6677
diff --git a/src/main/extension/extension_helper.cpp b/src/main/extension/extension_helper.cpp
67-
index c7b613226a..00885ee6a4 100644
78+
index 3aaf507faa..be084ef030 100644
6879
--- a/src/main/extension/extension_helper.cpp
6980
+++ b/src/main/extension/extension_helper.cpp
70-
@@ -326,7 +326,6 @@ vector<ExtensionUpdateResult> ExtensionHelper::UpdateExtensions(ClientContext &c
81+
@@ -337,7 +337,6 @@ vector<ExtensionUpdateResult> ExtensionHelper::UpdateExtensions(ClientContext &c
7182
vector<ExtensionUpdateResult> result;
7283
DatabaseInstance &db = DatabaseInstance::GetDatabase(context);
7384

7485
-#ifndef WASM_LOADABLE_EXTENSIONS
7586
case_insensitive_set_t seen_extensions;
7687

7788
// scan the install directory for installed extensions
78-
@@ -343,7 +342,6 @@ vector<ExtensionUpdateResult> ExtensionHelper::UpdateExtensions(ClientContext &c
89+
@@ -354,7 +353,6 @@ vector<ExtensionUpdateResult> ExtensionHelper::UpdateExtensions(ClientContext &c
7990

8091
result.push_back(UpdateExtensionInternal(context, db, fs, fs.JoinPath(ext_directory, path), extension_name));
8192
});
@@ -84,10 +95,10 @@ index c7b613226a..00885ee6a4 100644
8495
return result;
8596
}
8697
diff --git a/src/main/extension/extension_install.cpp b/src/main/extension/extension_install.cpp
87-
index e8ab595ab0..fb3e6371a3 100644
98+
index 2ea03b8e49..d8c710f153 100644
8899
--- a/src/main/extension/extension_install.cpp
89100
+++ b/src/main/extension/extension_install.cpp
90-
@@ -155,6 +155,9 @@ bool ExtensionHelper::CreateSuggestions(const string &extension_name, string &me
101+
@@ -145,6 +145,9 @@ bool ExtensionHelper::CreateSuggestions(const string &extension_name, string &me
91102
unique_ptr<ExtensionInstallInfo> ExtensionHelper::InstallExtension(DatabaseInstance &db, FileSystem &fs,
92103
const string &extension,
93104
ExtensionInstallOptions &options) {
@@ -97,7 +108,7 @@ index e8ab595ab0..fb3e6371a3 100644
97108
#ifdef WASM_LOADABLE_EXTENSIONS
98109
// Install is currently a no-op
99110
return nullptr;
100-
@@ -165,6 +168,9 @@ unique_ptr<ExtensionInstallInfo> ExtensionHelper::InstallExtension(DatabaseInsta
111+
@@ -155,6 +158,9 @@ unique_ptr<ExtensionInstallInfo> ExtensionHelper::InstallExtension(DatabaseInsta
101112

102113
unique_ptr<ExtensionInstallInfo> ExtensionHelper::InstallExtension(ClientContext &context, const string &extension,
103114
ExtensionInstallOptions &options) {
@@ -107,20 +118,11 @@ index e8ab595ab0..fb3e6371a3 100644
107118
#ifdef WASM_LOADABLE_EXTENSIONS
108119
// Install is currently a no-op
109120
return nullptr;
110-
@@ -209,7 +215,7 @@ string ExtensionHelper::ExtensionUrlTemplate(optional_ptr<const DatabaseInstance
111-
versioned_path = "/${REVISION}/${PLATFORM}/${NAME}.duckdb_extension";
112-
}
113-
#ifdef WASM_LOADABLE_EXTENSIONS
114-
- string default_endpoint = DEFAULT_REPOSITORY;
115-
+ string default_endpoint = ExtensionRepository::DEFAULT_REPOSITORY_URL;
116-
versioned_path = versioned_path + ".wasm";
117-
#else
118-
string default_endpoint = ExtensionRepository::DEFAULT_REPOSITORY_URL;
119121
diff --git a/src/main/extension/extension_load.cpp b/src/main/extension/extension_load.cpp
120-
index 84b28fef09..ea2ebba1cd 100644
122+
index 6e4bb18796..188931a84d 100644
121123
--- a/src/main/extension/extension_load.cpp
122124
+++ b/src/main/extension/extension_load.cpp
123-
@@ -328,7 +328,20 @@ bool ExtensionHelper::TryInitialLoad(DatabaseInstance &db, FileSystem &fs, const
125+
@@ -327,7 +327,20 @@ bool ExtensionHelper::TryInitialLoad(DatabaseInstance &db, FileSystem &fs, const
124126
direct_load = false;
125127
string extension_name = ApplyExtensionAlias(extension);
126128
#ifdef WASM_LOADABLE_EXTENSIONS
@@ -142,7 +144,7 @@ index 84b28fef09..ea2ebba1cd 100644
142144
string url = ExtensionFinalizeUrlTemplate(url_template, extension_name);
143145

144146
char *str = (char *)EM_ASM_PTR(
145-
@@ -369,35 +382,31 @@ bool ExtensionHelper::TryInitialLoad(DatabaseInstance &db, FileSystem &fs, const
147+
@@ -368,35 +381,31 @@ bool ExtensionHelper::TryInitialLoad(DatabaseInstance &db, FileSystem &fs, const
146148
direct_load = true;
147149
filename = fs.ExpandPath(filename);
148150
}
@@ -198,7 +200,7 @@ index 84b28fef09..ea2ebba1cd 100644
198200

199201
if (!metadata_mismatch_error.empty()) {
200202
throw InvalidInputException(metadata_mismatch_error);
201-
@@ -414,26 +423,192 @@ bool ExtensionHelper::TryInitialLoad(DatabaseInstance &db, FileSystem &fs, const
203+
@@ -413,26 +422,192 @@ bool ExtensionHelper::TryInitialLoad(DatabaseInstance &db, FileSystem &fs, const
202204
}
203205
}
204206

@@ -401,7 +403,7 @@ index 84b28fef09..ea2ebba1cd 100644
401403
#else
402404
auto dopen_from = filename;
403405
#endif
404-
@@ -449,28 +624,30 @@ bool ExtensionHelper::TryInitialLoad(DatabaseInstance &db, FileSystem &fs, const
406+
@@ -448,28 +623,30 @@ bool ExtensionHelper::TryInitialLoad(DatabaseInstance &db, FileSystem &fs, const
405407
result.filebase = lowercase_extension_name;
406408
result.filename = filename;
407409
result.lib_hdl = lib_hdl;

patches/duckdb/no_httpfs.patch

Lines changed: 0 additions & 44 deletions
This file was deleted.

0 commit comments

Comments
 (0)