Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor FilterableResolver to address #201 #283

Merged
merged 1 commit into from
Mar 22, 2023
Merged

Conversation

maheshrajamani
Copy link
Contributor

@maheshrajamani maheshrajamani commented Mar 22, 2023

What this PR does:
Refactored FilterableResolver to resolve to DBFilters.
Change the update and delete method to use FindOperation instead of ReadOperation
Removed getDocuments and getNewDocument from ReadOperation interface.
Refactored the test cases
Which issue(s) this PR fixes:
Fixes #201

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

…the command needed a read to resolve the document.

Addresses concern in #201
KEY,
/** Return only count of documents which satisfies the filter condition */
COUNT
KEY
Copy link
Contributor Author

@maheshrajamani maheshrajamani Mar 22, 2023

Choose a reason for hiding this comment

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

Holding on to this Enum, will be adding SORT_DOCUMENT type in the next PR for in memory sort.

@maheshrajamani maheshrajamani changed the title Refactored the code to use FindOperation instead of ReadOperation if … Refactor FilterableResolver to address #201 Mar 22, 2023
@maheshrajamani maheshrajamani marked this pull request as ready for review March 22, 2023 16:04
@maheshrajamani maheshrajamani requested a review from a team as a code owner March 22, 2023 16:04
Copy link
Contributor

@tatu-at-datastax tatu-at-datastax left a comment

Choose a reason for hiding this comment

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

LGTM

@maheshrajamani maheshrajamani merged commit 3a61c70 into main Mar 22, 2023
@maheshrajamani maheshrajamani deleted the find-refactor branch March 22, 2023 19:11
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.

Separate concerns in the ReadOperation
2 participants