Skip to content

Conversation

@ATL2001
Copy link
Contributor

@ATL2001 ATL2001 commented Nov 15, 2025

Per the discussion on discord, I'd like to add the spatial bench queries implemented using spatial polars. It's certainly not as fast as SedonaDB, and I'm not sure how long I'll keep it maintained once geopandas gets unblocked, (I expect that to easily eclipse what I've built, and when it gets rolling, I intend to help out on that project, and spatial polars will likely get left behind)

It's not completely 1:1 with the SQL based queries, but I believe I've stayed true to the intent. The KNN join that I implemented in spatial polars for Q12 only takes into account the centroids of the buildings, I'm not sure how the other engines approach that, but I felt like I was slightly cheating... 🙁

@jiayuasu
Copy link
Member

Great work!

Can you update the README as well to include Spatial polars?

@ATL2001
Copy link
Contributor Author

ATL2001 commented Nov 17, 2025

would you like me to add a little blurb to the README about spatial polars also not being SQL based similar to geopandas?

@MrPowers
Copy link
Contributor

@ATL2001 - this PR looks great and thanks for contributing!

@jiayuasu - looks like a reference was added to the README. Are any other changes needed or can we merge this?

@jiayuasu
Copy link
Member

@ATL2001 Can you please add this?

would you like me to add a little blurb to the README about spatial polars also not being SQL based similar to geopandas?

@ATL2001
Copy link
Contributor Author

ATL2001 commented Nov 20, 2025

I'll make some time this weekend to add to it. Given it's relative obscurity, are you open to me adding a link to the docs (googling spatial polars gives you results for geopolars and polars-ST).

@jiayuasu
Copy link
Member

I'll make some time this weekend to add to it. Given it's relative obscurity, are you open to me adding a link to the docs (googling spatial polars gives you results for geopolars and polars-ST).

@ATL2001 All sound good to me!

@ATL2001
Copy link
Contributor Author

ATL2001 commented Nov 22, 2025

I added a little section below the geopandas section for it, I didn't want to sound like a commercial, so I kept it to a minimum!

@jiayuasu jiayuasu merged commit 887eff4 into apache:main Nov 24, 2025
2 checks passed
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