Skip to content

feat: adding delete_by_filter and update_by_filter#2631

Merged
davidsbatista merged 9 commits into
mainfrom
feat/add-update-delete-by-filter-Astra
Dec 30, 2025
Merged

feat: adding delete_by_filter and update_by_filter#2631
davidsbatista merged 9 commits into
mainfrom
feat/add-update-delete-by-filter-Astra

Conversation

@davidsbatista
Copy link
Copy Markdown
Contributor

@davidsbatista davidsbatista commented Dec 29, 2025

Related Issues

Proposed Changes:

  • update_by_filter: updates documents matching filters using dot notation for nested meta fields (e.g., meta.status). New fields are created if missing via Astra's $set operator.
  • delete_by_filter: deletes documents matching the provided filters.
  • update_many to AstraClient: Supports bulk updates with proper return value handling.
  • added copyright headers to source files.
  • removed unused variable and outdated docstring parameter.

How did you test it?

  • added test_update_by_filter and test_delete_by_filter integration tests.

Checklist

@github-actions github-actions Bot added integration:astra type:documentation Improvements or additions to documentation labels Dec 29, 2025
@davidsbatista davidsbatista marked this pull request as ready for review December 30, 2025 08:57
@davidsbatista davidsbatista requested a review from a team as a code owner December 30, 2025 08:57
@davidsbatista davidsbatista requested review from sjrl and removed request for a team December 30, 2025 08:57
Copy link
Copy Markdown
Contributor

@sjrl sjrl 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! I'm not super familiar with Astra so I can't comment too much on the operations themselves but from what I can tell all good!

And I see better now that the astra_client is a convenience class that doesn't follow any sort of strict protocol or structure so feel free to disregard my comments on naming there.

@davidsbatista davidsbatista merged commit 38f1f1b into main Dec 30, 2025
10 checks passed
@davidsbatista davidsbatista deleted the feat/add-update-delete-by-filter-Astra branch December 30, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:astra type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add update_by_filter() and delete_by_filter() operations to AstraDocumentStore

2 participants