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

Implemented valueToString function used to generate the query identifier #13

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

ktsivkov
Copy link
Collaborator

  • Do only one thing
  • Non breaking API changes
  • Tested

What did this pull request do?

Implemented valueToString function for internal use.
Used to generate query identifier taking into account pointer passed values.
Ref #12

User Case Description

When the arguments for a query are passed by pointer they were previously hashed as pointers, now they are going to be addressed as values

…ier taking into account pointer-case query args

Co-authored-by: dennis-dko <[email protected]>
@ktsivkov ktsivkov self-assigned this Feb 22, 2024
@ktsivkov ktsivkov changed the title Implemented valueToString function used to generate the query identif… Implemented valueToString function used to generate the query identifier Feb 22, 2024
@ktsivkov ktsivkov merged commit 4e0b064 into fix-issue10 Feb 22, 2024
@ktsivkov ktsivkov deleted the fix-issue12 branch February 22, 2024 13:52
ktsivkov added a commit that referenced this pull request Feb 22, 2024
* Implemented cache invalidation handling, added a new required method in Cacher interface

* Implemented valueToString function used to generate the query identifier taking into account pointer-case query args (#13)

Co-authored-by: dennis-dko <[email protected]>

* renamed queryType constants, added tests for Caches.getMutator

Co-authored-by: dennis-dko <[email protected]>

---------

Co-authored-by: dennis-dko <[email protected]>
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.

None yet

1 participant