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

ISPN-2232 Make sure local query caches can also be registered #1272

Closed
wants to merge 1 commit into from

Conversation

galderz
Copy link
Member

@galderz galderz commented Aug 31, 2012

https://issues.jboss.org/browse/ISPN-2232

  • This can be achieved making sure the correct type is given when the query interceptor is retrieved from the component registry.
  • This avoids the need for unnecessary query interceptor registration because the custom interceptor chain handling already takes care of that, and with local indexing, you'd have QueryInterceptor.class component registered unnecessarily.

* This can be achieved making sure the correct type is given when the
query interceptor is retrieved from the component registry.
* This avoids the need for unnecessary query interceptor registration
because the custom interceptor chain handling already takes care of
that.
@mmarkus
Copy link
Contributor

mmarkus commented Aug 31, 2012

As the changes seem trivial and the suite is passing, we've decided to integrate this one into the release.
I'll leave this pull request open for Sanne to give a look just in case.

@ghost ghost assigned Sanne Aug 31, 2012
@Sanne
Copy link
Member

Sanne commented Aug 31, 2012

thanks! nice improvements :)

@Sanne Sanne closed this Aug 31, 2012
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.

3 participants