-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Allow member without JCache in classpath to join cluster when no Cache configurations are defined #9459
Conversation
Test PASSed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very nice!
…when no Cache configurations are defined
c1cddcc
to
5709318
Compare
Test PASSed. |
ICacheService cacheService = getService(); | ||
for (CacheConfig cacheConfig : configs) { | ||
cacheService.putCacheConfigIfAbsent(cacheConfig); | ||
if (isJcacheAvailable(getNodeEngine().getConfigClassLoader())) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isJCacheAvailable
with capital C (it is not introduced here but just wanted to note)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ahmetmircik , I renamed the methods in JCacheDetector
with a capital C in a separate commit. Once green will squash & merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor comment
Test PASSed. |
Allows a member without JCache API in its classpath to join a cluster whose members do have the JCache API, when JCache is not used (no
CacheConfig
s are already defined) and log a warning that if JCache API will be used, it will fail.Also prints an informative
SEVERE
message in caseCacheConfig
s are delivered in thePostJoinCacheOperation
but JCache API artifact is missing from the classpath (currently, a service not found exception is logged that indicates thatCacheService
is not available but does not provide information towards resolving the issue).Fixes #9301