Skip to content

DATAKV-137 - Fix cached query execution. #20

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

Closed
wants to merge 2 commits into from
Closed

Conversation

mp911de
Copy link
Member

@mp911de mp911de commented Jun 6, 2016

We now make sure that cached SpelCriteria's use the appropriate EvaluationContext containing parameters from the request instead of using the EvaluationContext of the cached query.


Related ticket: DATAKV-137

mp911de added 2 commits June 6, 2016 13:23
We now make sure that cached SpelCriteria's use the appropriate EvaluationContext containing parameters from the request instead of using the EvaluationContext of the cached query.
@mp911de mp911de force-pushed the issue/DATAKV-137 branch from e9f8b62 to b3debff Compare June 6, 2016 11:24
christophstrobl pushed a commit that referenced this pull request Jun 14, 2016
We now make sure that cached SpelCriteria's use the appropriate EvaluationContext containing parameters from the request instead of using the EvaluationContext of the cached query.

Original Pull Request: #20
christophstrobl added a commit that referenced this pull request Jun 14, 2016
Remove whitespaces & unused imports, fix warnings.

Original Pull Request: #20
christophstrobl pushed a commit that referenced this pull request Jun 14, 2016
We now make sure that cached SpelCriteria's use the appropriate EvaluationContext containing parameters from the request instead of using the EvaluationContext of the cached query.

Original Pull Request: #20
christophstrobl added a commit that referenced this pull request Jun 14, 2016
Remove whitespaces & unused imports, fix warnings.

Original Pull Request: #20
@christophstrobl christophstrobl deleted the issue/DATAKV-137 branch June 14, 2016 06:26
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.

2 participants