Update duckdb to get Latest Relation

* Update Workflow definition to use fix commit for duckdb submodule * Fix extension code required because of update of duckdb
substrait-io · Nov 13, 2024 · b1a4fd6 · b1a4fd6
1 parent 82bad97
commit b1a4fd6
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/main_distribution.yml b/.github/workflows/main_distribution.yml
@@ -21,7 +21,7 @@ jobs:
     name: Build extension binaries
     uses: duckdb/extension-ci-tools/.github/workflows/_extension_distribution.yml@main
     with:
-      duckdb_version: main
+      duckdb_version: ca5af32c331f9d5ea49f7158d5c83a47f25b8b79
       ci_tools_version: main
       exclude_archs: "wasm_mvp;wasm_eh;wasm_threads;windows_amd64;windows_amd64_mingw"
       extension_name: substrait

diff --git a/duckdb b/duckdb
diff --git a/src/to_substrait.cpp b/src/to_substrait.cpp
@@ -1317,7 +1317,7 @@ substrait::Rel *DuckDBToSubstrait::TransformGet(LogicalOperator &dop) {
 		auto &column_ids = dget.GetColumnIds();
 		for (auto col_idx : dget.projection_ids) {
 			auto struct_item = select->add_struct_items();
-			struct_item->set_field(static_cast<int32_t>(column_ids[col_idx]));
+			struct_item->set_field(static_cast<int32_t>(column_ids[col_idx].GetPrimaryIndex()));
 			// FIXME do we need to set the child? if yes, to what?
 		}
 		projection->set_allocated_select(select);