Skip to content

feat: add DeleteResult with num_deleted_rows#6001

Merged
wjones127 merged 3 commits intolance-format:mainfrom
wkalt:task/add-num-deleted-rows-delete
Feb 24, 2026
Merged

feat: add DeleteResult with num_deleted_rows#6001
wjones127 merged 3 commits intolance-format:mainfrom
wkalt:task/add-num-deleted-rows-delete

Conversation

@wkalt
Copy link
Copy Markdown
Contributor

@wkalt wkalt commented Feb 24, 2026

This introduces a DeleteResult with num_rows_deleted, similar to UpdateResult.

This introduces a DeleteResult with num_rows_deleted, similar to
UpdateResult.
@github-actions github-actions Bot added enhancement New feature or request python labels Feb 24, 2026
Copy link
Copy Markdown
Contributor

@wjones127 wjones127 left a comment

Choose a reason for hiding this comment

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

One small change needed, otherwise seems good.

Comment thread rust/lance/src/dataset/write/delete.rs
Co-authored-by: Will Jones <willjones127@gmail.com>
@wkalt
Copy link
Copy Markdown
Contributor Author

wkalt commented Feb 24, 2026

@wjones127 thanks, committed your suggestion

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 24, 2026

Codecov Report

❌ Patch coverage is 81.35593% with 11 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
rust/lance/src/dataset/write/delete.rs 80.35% 5 Missing and 6 partials ⚠️

📢 Thoughts on this report? Let us know!

@wjones127 wjones127 merged commit 914e275 into lance-format:main Feb 24, 2026
29 checks passed
wjones127 added a commit to wjones127/lance that referenced this pull request Feb 25, 2026
This introduces a DeleteResult with num_rows_deleted, similar to
UpdateResult.

---------

Co-authored-by: Will Jones <willjones127@gmail.com>
wjones127 added a commit to wjones127/lance that referenced this pull request Feb 25, 2026
This introduces a DeleteResult with num_rows_deleted, similar to
UpdateResult.

---------

Co-authored-by: Will Jones <willjones127@gmail.com>
wjones127 added a commit that referenced this pull request Feb 26, 2026
This introduces a DeleteResult with num_rows_deleted, similar to
UpdateResult.

---------

Co-authored-by: Will Jones <willjones127@gmail.com>
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