Skip to content
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

docs(duckdb-geospatial): document adding R-Tree spatial index with raw_sql #10097

Open
1 task done
ncclementi opened this issue Sep 11, 2024 · 3 comments
Open
1 task done
Labels
duckdb The DuckDB backend feature Features or general enhancements geospatial Geospatial related functionality

Comments

@ncclementi
Copy link
Contributor

Is your feature request related to a problem?

No

What is the motivation behind your request?

DuckDB added support for spatial indexes in their latest release 1.1.0 which will enable faster spatial joins.
See https://duckdb.org/2024/09/09/announcing-duckdb-110.html#r-tree

Describe the solution you'd like

I'd like Ibis to have a way to support the creation of this type of index. I'm not sure if this is possible but I'd like to discuss how we would implement this in the comments of this issue.

What version of ibis are you running?

main

What backend(s) are you using, if any?

DuckDB

Code of Conduct

  • I agree to follow this project's Code of Conduct
@ncclementi ncclementi added the feature Features or general enhancements label Sep 11, 2024
@ncclementi ncclementi added geospatial Geospatial related functionality duckdb The DuckDB backend labels Sep 11, 2024
@gforsyth
Copy link
Member

As per some internal discussion, we may not want to open up the huge can of worms that represents support for creating indexes, but we should, at the least, document how a user can go about both creating those indexes using raw_sql and also checking whether an existing table HAS an index on it (as a pre-check before doing an otherwise more expensive spatial join)

@ncclementi
Copy link
Contributor Author

I'm happy to close this as not planed, unless we want to leave it open for some reason.

@gforsyth
Copy link
Member

I would leave it open for now and close it out with a PR adding a how-to doc on using raw_sql to add the index and how to check that an rtree index has been added.

@gforsyth gforsyth changed the title feat(duckdb-geospatial): Add support for R-Tree spatial index docs(duckdb-geospatial): document adding R-Tree spatial index with raw_sql Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duckdb The DuckDB backend feature Features or general enhancements geospatial Geospatial related functionality
Projects
Status: backlog
Development

No branches or pull requests

2 participants