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

Use cached deserialized values in SQL when possible [IMDG-125] #18172

Merged
merged 6 commits into from Feb 11, 2021
Merged

Use cached deserialized values in SQL when possible [IMDG-125] #18172

merged 6 commits into from Feb 11, 2021

Conversation

devozerov
Copy link
Contributor

@devozerov devozerov commented Feb 9, 2021

This PR ensures that the SQL engine attempts to re-use already deserialized values stored in an index, similarly to predicate API. This substantially improves map scan operations performance when there is at least one index on an IMap.

In addition to this, the PR ensures that we move both serialized and deserialized representations to the SQL engine to avoid serialization of entries that already have a Data representation. This improves the performance of index lookups by ~10-15%.

@devozerov devozerov self-assigned this Feb 9, 2021
@devozerov devozerov added this to the 4.2 milestone Feb 9, 2021
@devozerov devozerov marked this pull request as draft February 9, 2021 09:09
@hazelcast hazelcast deleted a comment from hz-devops-test Feb 9, 2021
@devozerov devozerov marked this pull request as ready for review February 9, 2021 13:16
@devozerov devozerov requested review from hakanmemisoglu and removed request for petrpleshachkov February 10, 2021 16:02
@devozerov devozerov merged commit e891589 into hazelcast:master Feb 11, 2021
@devozerov devozerov deleted the sql-cached-index-value branch February 11, 2021 09:26
@devozerov devozerov changed the title Use cached deserialized values in SQL when possible Use cached deserialized values in SQL when possible [IMDG-125] Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Source: Internal PR or issue was opened by an employee Team: SQL Type: Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants