Skip to content

Fix NPE in declarative aggregation execution. #3625

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

Conversation

christophstrobl
Copy link
Member

This PR fixes an issue where using a simple return type leads to NPE when the actual aggregation result does not contain any values.

Closes: #3623

This commit fixes an issue where using a simple return type leads to NPE when the actual aggregation result does not contain any values.

Closes: #3623
@mp911de mp911de added the type: bug A general bug label Apr 13, 2021
@mp911de mp911de added this to the 3.0.9 (Neumann SR9) milestone Apr 13, 2021
mp911de pushed a commit that referenced this pull request Apr 13, 2021
This commit fixes an issue where using a simple return type leads to NPE when the actual aggregation result does not contain any values.

Closes: #3623
Original pull request: #3625.
mp911de added a commit that referenced this pull request Apr 13, 2021
Use ObjectUtils for empty check.

See #3623
Original pull request: #3625.
mp911de pushed a commit that referenced this pull request Apr 13, 2021
This commit fixes an issue where using a simple return type leads to NPE when the actual aggregation result does not contain any values.

Closes: #3623
Original pull request: #3625.
mp911de added a commit that referenced this pull request Apr 13, 2021
Use ObjectUtils for empty check.

See #3623
Original pull request: #3625.
mp911de pushed a commit that referenced this pull request Apr 13, 2021
This commit fixes an issue where using a simple return type leads to NPE when the actual aggregation result does not contain any values.

Closes: #3623
Original pull request: #3625.
mp911de added a commit that referenced this pull request Apr 13, 2021
Use ObjectUtils for empty check.

See #3623
Original pull request: #3625.
@mp911de
Copy link
Member

mp911de commented Apr 13, 2021

That's merged, polished, and backported now.

@mp911de mp911de closed this Apr 13, 2021
@mp911de mp911de deleted the issue/3623 branch April 13, 2021 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@Aggregation repository query method causes NullPointerException when the result is empty
2 participants