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

Throw CacheNotExistsException when cache config does not exist #12593

Merged

Conversation

@mmedenjak
Copy link
Contributor

@mmedenjak mmedenjak commented Mar 14, 2018

Avoids throwing a NullPointerException and throws a
CacheNotExistsException instead when the journal is being created but
the cache config does not exist.

Fixes: hazelcast/hazelcast-enterprise#1980

Avoids throwing a NullPointerException and throws a
CacheNotExistsException instead when the journal is being created but
the cache config does not exist.

Fixes: hazelcast/hazelcast-enterprise#1980
@mmedenjak mmedenjak added this to the 3.10 milestone Mar 14, 2018
@mmedenjak mmedenjak self-assigned this Mar 14, 2018
@mmedenjak mmedenjak requested review from mdogan and Donnerbart Mar 14, 2018
@mdogan
mdogan approved these changes Mar 14, 2018
@mmedenjak
Copy link
Contributor Author

@mmedenjak mmedenjak commented Mar 15, 2018

verify

@devOpsHazelcast
Copy link
Contributor

@devOpsHazelcast devOpsHazelcast commented Mar 15, 2018

Test PASSed.

@mmedenjak mmedenjak merged commit 265cf3a into hazelcast:master Mar 15, 2018
1 check passed
1 check passed
default Build finished.
Details
@mmedenjak mmedenjak deleted the mmedenjak:journal-cache-not-exists-check branch Mar 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.