Skip to content

feat: expose sql api for python dataset#4267

Merged
jackye1995 merged 1 commit intolance-format:mainfrom
Jay-ju:python_sql
Jul 21, 2025
Merged

feat: expose sql api for python dataset#4267
jackye1995 merged 1 commit intolance-format:mainfrom
Jay-ju:python_sql

Conversation

@Jay-ju
Copy link
Copy Markdown
Contributor

@Jay-ju Jay-ju commented Jul 20, 2025

No description provided.

@Jay-ju Jay-ju marked this pull request as draft July 20, 2025 01:04
@github-actions github-actions Bot added enhancement New feature or request python labels Jul 20, 2025
@Jay-ju Jay-ju force-pushed the python_sql branch 2 times, most recently from 5b0df92 to 0809c00 Compare July 21, 2025 01:16
@Jay-ju Jay-ju marked this pull request as ready for review July 21, 2025 01:17
@Jay-ju Jay-ju force-pushed the python_sql branch 4 times, most recently from 10bddf7 to c964f37 Compare July 21, 2025 02:01
@Jay-ju
Copy link
Copy Markdown
Contributor Author

Jay-ju commented Jul 21, 2025

@jackye1995 This PR mainly opens the SQL interface to the Python module. Personally, I think SQL is a quite useful function, so I made it exposed. Please take a look when you have time.

@Jay-ju Jay-ju force-pushed the python_sql branch 2 times, most recently from 238ad47 to 6785f90 Compare July 21, 2025 03:52
Comment thread python/python/lance/dataset.py Outdated
Comment thread src/dataset.rs Outdated
@Jay-ju Jay-ju force-pushed the python_sql branch 2 times, most recently from 10a4671 to 5ffd564 Compare July 21, 2025 11:49
Copy link
Copy Markdown
Contributor

@jackye1995 jackye1995 left a comment

Choose a reason for hiding this comment

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

looks good to me!

@jackye1995 jackye1995 merged commit 40f63b6 into lance-format:main Jul 21, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants