Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ testext
test/python/__pycache__/
.Rhistory
compile_commands.json
vortex-extension/target/

#proto gen folder
gen/
gen/
3 changes: 0 additions & 3 deletions .gitmodules
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,3 @@
[submodule "vcpkg"]
path = vcpkg
url = https://github.com/microsoft/vcpkg.git
[submodule "vortex"]
path = vortex
url = https://github.com/vortex-data/vortex.git
6 changes: 3 additions & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,8 @@ if(EXISTS "/.dockerenv")
endif()

corrosion_import_crate(
MANIFEST_PATH vortex/Cargo.toml
CRATES vortex-duckdb
MANIFEST_PATH vortex-extension/Cargo.toml
CRATES vortex-extension
CRATE_TYPES ${VORTEX_LIB_TYPE}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this still work with shared vortex build?

FLAGS --crate-type=${VORTEX_LIB_TYPE}
)
Expand All @@ -82,7 +82,7 @@ corrosion_set_env_vars(
"DUCKDB_SOURCE_DIR=${DUCKDB_MODULE_BASE_DIR}"
)

include_directories(src/include vortex/vortex-duckdb/include vortex/vortex-duckdb/cpp/include)
include_directories(src/include)

set(EXTENSION_NAME ${TARGET_NAME}_extension)
set(LOADABLE_EXTENSION_NAME ${TARGET_NAME}_loadable_extension)
Expand Down
10 changes: 10 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -158,3 +158,13 @@ lldb -- ./build/debug/duckdb -unsigned
When a PR is merged to `main`, the cherry-pick workflow automatically applies the squash commit to the current release branch.

The current release branch is configured in `.github/workflows/CherryPick.yml` via the `RELEASE_BRANCH` env var.

## Changing Vortex version

The Vortex version is defined in `vortex-extension/Cargo.toml`. It can be a git commit, tag, branch or even a local path:

```toml
vortex-duckdb = { path = "<path/to/vortex/vortex-duckdb`>}
```

See the Cargo docs for [git](https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-git-repositories) or [path](https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-path-dependencies) dependencies for full details.
1 change: 0 additions & 1 deletion rust-toolchain.toml

This file was deleted.

5 changes: 5 additions & 0 deletions rust-toolchain.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
[toolchain]
channel = "1.91.0"
components = ["rust-src", "rustfmt", "clippy", "rust-analyzer"]
profile = "minimal"

9 changes: 9 additions & 0 deletions src/include/vortex_rust.hpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
#pragma once

extern "C" {

void vortex_init_rust(void *db);
const char *vortex_version_rust();
const char *vortex_extension_version_rust();

}
3 changes: 1 addition & 2 deletions src/vortex_extension.cpp
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
#define DUCKDB_EXTENSION_MAIN

#include "vortex_extension.hpp"
#include "duckdb_vx/table_function.h"
#include "vortex.h"
#include "vortex_rust.hpp"

using namespace duckdb;

Expand Down
1 change: 0 additions & 1 deletion vortex
Submodule vortex deleted from 7b5362
Loading
Loading