Update submodules to DuckDB 8616efa9da

Author: staticlibs

duckdb

@@ -35,7 +35,7 @@ class PostgresBinaryCopyFunction : public CopyFunction {
 	                                        GlobalFunctionData &gstate);
 
 	static unique_ptr<FunctionData> PostgresBinaryReadBind(ClientContext &context, CopyFromFunctionBindInput &info,
-	                                                       vector<string> &expected_names,
+	                                                       vector<Identifier> &expected_names,
 	                                                       vector<LogicalType> &expected_types);
 };
 

extension-ci-tools

@@ -47,8 +47,8 @@ class PostgresCatalog : public Catalog {
 	string GetCatalogType() override {
 		return "postgres";
 	}
-	Identifier GetDefaultSchema() const override {
-		return default_schema.empty() ? "public" : default_schema;
+	optional<Identifier> GetDefaultSchema() const override {
+		return default_schema.empty() ? Identifier("public") : default_schema;
 	}
 
 	string GetConnectionString();

src/include/postgres_binary_copy.hpp

@@ -98,7 +98,7 @@ void PostgresBinaryCopyFunction::PostgresBinaryWriteFinalize(ClientContext &cont
 
 struct PostgresBinaryReadBindData : public TableFunctionData {
 	string file_path;
-	vector<string> names;
+	vector<Identifier> names;
 	vector<LogicalType> types;
 	vector<PostgresType> postgres_types;
 	idx_t buffer_size = PostgresBinaryFileReader::DEFAULT_BUFFER_SIZE;
@@ -125,7 +125,7 @@ struct PostgresBinaryReadGlobalState : public GlobalTableFunctionState {
 
 unique_ptr<FunctionData> PostgresBinaryCopyFunction::PostgresBinaryReadBind(ClientContext &context,
                                                                             CopyFromFunctionBindInput &info,
-                                                                            vector<string> &expected_names,
+                                                                            vector<Identifier> &expected_names,
                                                                             vector<LogicalType> &expected_types) {
 	auto result = make_uniq<PostgresBinaryReadBindData>();
 	result->file_path = info.info.file_path;
@@ -178,7 +178,7 @@ static unique_ptr<FunctionData> ReadPostgresBinaryBind(ClientContext &context, T
 		result->postgres_types.push_back(PostgresUtils::CreateEmptyPostgresType(col_type));
 	}
 
-	result->names = IdentifiersToStrings(names);
+	result->names = names;
 	result->types = return_types;
 
 	if (input.named_parameters.count("buffer_size")) {

src/include/storage/postgres_catalog.hpp

@@ -12,7 +12,7 @@ require postgres_scanner
 statement error
 ATTACH 'dbname=nope' AS d (TYPE POSTGRES, CONNECT_DISPLAY NULL)
 ----
-NULL is not supported as a valid option for ATTACH option "connect_display"
+NULL is not supported as a valid option for ATTACH option
 
 # the value ends up in the shell prompt verbatim, so escape sequences must not survive
 statement error

src/postgres_binary_copy.cpp

@@ -45,7 +45,7 @@ DELETE FROM v1
 statement error
 INSERT INTO v1 VALUES (1, 1);
 ----
-table v1 has 1 columns but 2 values were supplied
+has 1 columns but 2 values were supplied
 
 statement ok
 CREATE VIEW IF NOT EXISTS v1 AS SELECT 84;