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

[query-cache] QueryCache on an IMap that is subsequently destroyed and recreated doesnt work #11666

Closed
vertex-github opened this issue Oct 25, 2017 · 4 comments

Comments

@vertex-github
Copy link

@vertex-github vertex-github commented Oct 25, 2017

Create a QueryCache on an IMap (which triggers the IMap creation).
Destroy the IMap.
Get the IMap by name again.
Get the QueryCache from the IMap.

HZ returns a seemingly valid QueryCache.

Update the IMap.

The update event never gets fired to the QueryCache.

I expect either the QueryCache to get the event, or the QueryCache should be destroyed when the 1st IMap is destroyed, and any attempts to retrieve it after that fact should result in a null

@mmedenjak mmedenjak changed the title QueryCache on an IMap that is subsequently destroyed and recreated doesnt work [query-cache] QueryCache on an IMap that is subsequently destroyed and recreated doesnt work Oct 26, 2017
@mmedenjak mmedenjak added this to the 3.9.1 milestone Oct 26, 2017
@ahmetmircik
Copy link
Member

@ahmetmircik ahmetmircik commented Oct 30, 2017

@vertex-github Please try with 3.8.6, this seems fixed there, if not, we can re-open this issue.

@vertex-github
Copy link
Author

@vertex-github vertex-github commented Oct 31, 2017

Can you point to a changeset between 3.8.1 and 3.8.6 that fixes this?

@ahmetmircik
Copy link
Member

@ahmetmircik ahmetmircik commented Oct 31, 2017

It should be this according to my tests: #10168

Did you have a chance to try with 3.8.6?

@vertex-github
Copy link
Author

@vertex-github vertex-github commented Nov 1, 2017

OK, I tested with 3.8.6 (along with all versions in between 3.8.1 and up). The behavior is different with 3.8.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.