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-11674 RemoteCache cleanup #8216
ISPN-11674 RemoteCache cleanup #8216
Conversation
6722ebd
to
08fb27c
Compare
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 comments.
client/hotrod-client/src/main/java/org/infinispan/client/hotrod/RemoteCacheManager.java
Outdated
Show resolved
Hide resolved
client/hotrod-client/src/main/java/org/infinispan/client/hotrod/impl/DelegatingRemoteCache.java
Outdated
Show resolved
Hide resolved
...otrod-client/src/main/java/org/infinispan/client/hotrod/impl/InvalidatedNearRemoteCache.java
Outdated
Show resolved
Hide resolved
client/hotrod-client/src/main/java/org/infinispan/client/hotrod/impl/RemoteCacheEntrySet.java
Outdated
Show resolved
Hide resolved
client/hotrod-client/src/main/java/org/infinispan/client/hotrod/impl/RemoteCacheEntrySet.java
Outdated
Show resolved
Hide resolved
client/hotrod-client/src/test/java/org/infinispan/client/hotrod/CacheManagerNotStartedTest.java
Outdated
Show resolved
Hide resolved
client/hotrod-client/src/test/java/org/infinispan/client/hotrod/CacheManagerStoppedTest.java
Outdated
Show resolved
Hide resolved
client/hotrod-client/src/test/java/org/infinispan/client/hotrod/SecureExecTest.java
Outdated
Show resolved
Hide resolved
client/hotrod-client/src/test/java/org/infinispan/client/hotrod/retry/ReplicationRetryTest.java
Outdated
Show resolved
Hide resolved
client/hotrod-client/src/main/java/org/infinispan/client/hotrod/impl/RemoteCacheKeySet.java
Outdated
Show resolved
Hide resolved
compile error in CI. cc @wburns |
Yeah, sorry @pruivo I haven't cleaned this one up yet since your review. |
08fb27c
to
4d15aa3
Compare
Okay I have addressed all the review comments and made sure it compiles. Here is to it passing CI :) |
@wburns can you check CI? JCache tests are failing, some of them with ps. I'll re-review the changes later :) |
...rc/main/java/org/infinispan/client/hotrod/impl/transaction/TransactionalRemoteCacheImpl.java
Outdated
Show resolved
Hide resolved
jcache/remote/src/main/java/org/infinispan/jcache/remote/RemoteCacheWrapper.java
Show resolved
Hide resolved
jcache/remote/src/main/java/org/infinispan/jcache/remote/RemoteCacheWithStats.java
Outdated
Show resolved
Hide resolved
4d15aa3
to
c509698
Compare
Updated again :) |
CI seems to be okay. Just rerunning to rule out the SCRAM failure, which should be unrelated. |
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.
looks good to me 👍 just a minor "nitpick"
jcache/remote/src/main/java/org/infinispan/jcache/remote/RemoteCacheWithStats.java
Outdated
Show resolved
Hide resolved
c509698
to
e2a1cf1
Compare
Updated |
integrated! thanks @wburns ! |
https://issues.redhat.com/browse/ISPN-11674
https://issues.redhat.com/browse/ISPN-11619