8000 add ducklake to internal_extensions by c-herrewijn · Pull Request #17811 · duckdb/duckdb · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

add ducklake to internal_extensions #17811

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

c-herrewijn
Copy link
Member
@c-herrewijn c-herrewijn commented Jun 5, 2025

related PR: #17789

  • adds ducklake to output of duckdb_extensions()
  • makes ducklake auto-installable and auto-loadable

8000
@c-herrewijn c-herrewijn requested a review from Maxxen June 5, 2025 12:02
@duckdb-draftbot duckdb-draftbot marked this pull request as draft June 5, 2025 16:05
@c-herrewijn c-herrewijn mentioned this pull request Jun 5, 2025
@c-herrewijn c-herrewijn marked this pull request as ready for review June 5, 2025 16:47
Mytherin added a commit that referenced this pull request Jun 10, 2025
continuation of #17811

This PR makes it possible to install ducklake for a duckdb development
build with:

```bash
CORE_EXTENSIONS="ducklake" make
./build/release/duckdb -unsigned
```

```sql
INSTALL ducklake from 'build/release/repository';
LOAD ducklake;
```

Instead of using `LOAD ducklake;`, the ducklake extension can also be
auto-loaded, by using one of its functions, e.g. with:
```sql
ATTACH 'ducklake:metadata.ducklake' AS my_ducklake;
```

**NOTE**:
for function `duckdb_extension_load()`, flag `LOAD_TESTS` has been
omitted on purpose, since there are failing tests. This needs to be
added back at a later moment.

Failing test
[ducklake_extension_fc-src/test/sql/autoloading/autoload_data_path.test](https://github.com/duckdb/ducklake/blob/main/test/sql/autoloading/autoload_data_path.test)
see:
https://github.com/duckdb/duckdb/actions/runs/15443909213/job/43472610082#step:6:17666
@Mytherin Mytherin merged commit 6638536 into duckdb:v1.3-ossivalis Jun 10, 2025
40 of 50 checks passed
@c-herrewijn c-herrewijn deleted the add-ducklake-to-duckdb-extensions branch June 10, 2025 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0