Merge pull request #67 from carlopi/absorb_patches

Absorb patches, move to duckdb's main

Author: Mytherin

.github/workflows/MainDistributionPipeline.yml

@@ -17,26 +17,17 @@ jobs:
     uses: duckdb/extension-ci-tools/.github/workflows/_extension_distribution.yml@main
     with:
       extension_name: mysql_scanner
-      duckdb_version: 'v0.10.2'
-      exclude_archs: 'osx_amd64;osx_arm64;wasm_mvp;wasm_eh;wasm_threads;windows_amd64_rtools'
+      duckdb_version: main
+      exclude_archs: 'wasm_mvp;wasm_eh;wasm_threads;windows_amd64_rtools'
       build_duckdb_shell: false
 
-  # Note: this workaround is required for building MacOS where extra toolchain config is required
-  duckdb-stable-build-macos:
-    name: Build extension binaries
-    uses: ./.github/workflows/_extension_distribution_macos.yml
-    with:
-      duckdb_version: v0.10.2
-      extension_name: mysql_scanner
-
-
   duckdb-stable-deploy:
     name: Deploy extension binaries
-    needs: [duckdb-stable-build, duckdb-stable-build-macos]
+    needs: duckdb-stable-build
     uses: duckdb/extension-ci-tools/.github/workflows/_extension_deploy.yml@main
     secrets: inherit
     with:
-      duckdb_version: v0.10.2
+      duckdb_version: main
       extension_name: mysql_scanner
       exclude_archs: 'wasm_mvp;wasm_eh;wasm_threads;windows_amd64_rtools'
       deploy_latest: ${{ startsWith(github.ref, 'refs/tags/v') || github.ref == 'refs/heads/main' }}

.github/workflows/MysqlTests.yml

@@ -54,7 +54,7 @@ jobs:
 
       - uses: actions/setup-python@v2
         with:
-          python-version: '3.7'
+          python-version: '3.11'
 
       - name: Setup vcpkg
         uses: lukka/[email protected]
@@ -70,9 +70,6 @@ jobs:
       - name: Build extension
         shell: bash
         run: |
-          # needed for MySQL cross compilation - see https://mariadb.com/kb/en/cross-compiling-mariadb/
-          echo 'SET(STACK_DIRECTION -1)' >> ${VCPKG_TOOLCHAIN_PATH}
-          echo 'SET(HAVE_IB_GCC_ATOMIC_BUILTINS 0)' >> ${VCPKG_TOOLCHAIN_PATH}
           make release
 
       - name: Test Extension

.github/workflows/_extension_distribution_macos.yml

@@ -8,6 +8,7 @@
 
 #pragma once
 
+#include "duckdb/common/shared_ptr.hpp"
 #include "mysql_utils.hpp"
 #include "mysql_result.hpp"
 

duckdb

@@ -8,7 +8,7 @@
 
 using namespace duckdb;
 
-class MySQLScannerExtension : public Extension {
+class MysqlScannerExtension : public Extension {
 public:
 	std::string Name() override {
 		return "mysql_scanner";

src/include/mysql_connection.hpp

@@ -44,7 +44,7 @@ class MySQLTableEntry : public TableCatalogEntry {
 
 	TableStorageInfo GetStorageInfo(ClientContext &context) override;
 
-	void BindUpdateConstraints(LogicalGet &get, LogicalProjection &proj, LogicalUpdate &update,
+	void BindUpdateConstraints(Binder &binder, LogicalGet &get, LogicalProjection &proj, LogicalUpdate &update,
 	                           ClientContext &context) override;
 };
 

src/include/mysql_scanner_extension.hpp

@@ -28,7 +28,7 @@ MySQLConnection &MySQLConnection::operator=(MySQLConnection &&other) noexcept {
 
 MySQLConnection MySQLConnection::Open(const string &connection_string) {
 	MySQLConnection result;
-	result.connection = make_shared<OwnedMySQLConnection>(MySQLUtils::Connect(connection_string));
+	result.connection = make_shared_ptr<OwnedMySQLConnection>(MySQLUtils::Connect(connection_string));
 	result.dsn = connection_string;
 	result.Execute("SET character_set_results = 'utf8mb4';");
 	return result;

src/include/storage/mysql_table_entry.hpp

@@ -97,7 +97,7 @@ static void LoadInternal(DatabaseInstance &db) {
 	config.optimizer_extensions.push_back(std::move(mysql_optimizer));
 }
 
-void MySQLScannerExtension::Load(DuckDB &db) {
+void MysqlScannerExtension::Load(DuckDB &db) {
 	LoadInternal(*db.instance);
 }
 

src/mysql_connection.cpp

@@ -81,7 +81,7 @@ unique_ptr<BaseStatistics> MySQLTableEntry::GetStatistics(ClientContext &context
 	return nullptr;
 }
 
-void MySQLTableEntry::BindUpdateConstraints(LogicalGet &, LogicalProjection &, LogicalUpdate &, ClientContext &) {
+void MySQLTableEntry::BindUpdateConstraints(Binder &binder, LogicalGet &, LogicalProjection &, LogicalUpdate &, ClientContext &) {
 }
 
 TableFunction MySQLTableEntry::GetScanFunction(ClientContext &context, unique_ptr<FunctionData> &bind_data) {

src/mysql_extension.cpp

@@ -4,7 +4,9 @@
 
 require mysql_scanner
 
-require-env MYSQL_TEST_DATABASE_AVAILABLErequire-env POSTGRES_TEST_DATABASE_AVAILABLE
+require-env MYSQL_TEST_DATABASE_AVAILABLE
+
+require-env POSTGRES_TEST_DATABASE_AVAILABLE
 
 statement ok
 ATTACH 'host=localhost user=root port=0 database=mysqlscanner' AS s (TYPE MYSQL_SCANNER)

src/storage/mysql_table_entry.cpp

@@ -22,22 +22,18 @@ endif()
 
 set(CROSS_COMPILING "")
 set(STACK_DIRECTION "")
-if(VCPKG_TARGET_ARCHITECTURE STREQUAL "x86" OR VCPKG_TARGET_ARCHITECTURE STREQUAL "x64")
+if(NOT VCPKG_TARGET_IS_WINDOWS)
     set(STACK_DIRECTION -DSTACK_DIRECTION=-1)
-else()
-    # ARM builds are always cross compiled
+    set(HAVE_IB_GCC_ATOMIC_BUILTINS 0)
+    set(CROSS_COMPILING -DCMAKE_CROSSCOMPILING=1)
+    # Non-Windows builds are always cross-compiled
     # as such we build the executables (comp_sql, uca9dump, comp_client_err) separately
     set(PATCH_FILES
         ${PATCH_FILES}
         remove_executables.patch
     )
-    if(VCPKG_TARGET_IS_LINUX)
-        set(CROSS_COMPILING -DCMAKE_CROSSCOMPILING=1)
-        set(STACK_DIRECTION -DSTACK_DIRECTION=-1)
-    endif()
 endif()
 
-
 vcpkg_from_github(
     OUT_SOURCE_PATH SOURCE_PATH
     REPO mysql/mysql-server