Skip to content

fix(namespace): remove trailing empty query in table uri#6415

Merged
wjones127 merged 1 commit into
lance-format:mainfrom
jackye1995:remove-uri-query
Apr 8, 2026
Merged

fix(namespace): remove trailing empty query in table uri#6415
wjones127 merged 1 commit into
lance-format:mainfrom
jackye1995:remove-uri-query

Conversation

@jackye1995
Copy link
Copy Markdown
Contributor

Before this fix, it is possible for table uri to have a trailing ? because we do not clear the query component.

@github-actions github-actions Bot added the bug Something isn't working label Apr 6, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 6, 2026

Codecov Report

❌ Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
rust/lance-namespace-impls/src/dir/manifest.rs 88.88% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@wjones127 wjones127 merged commit a7aba1f into lance-format:main Apr 8, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants