jdeppe-pivotal commented on pull request #6447:
URL: https://github.com/apache/geode/pull/6447#issuecomment-834735135


   > It doesn't look like `KeysNativeRedisAcceptanceTest` or 
`ScanNativeRedisAcceptanceTest` got switched over to use the 
`NativeRedisClusterTestRule`. We also didn't edit `AbstractKeysIntegrationTest` 
to use a `JedisCluster` in addition to `AbstractScanIntegrationTest`. Did we 
not switch the `AbstractKeysIntegrationTest` and 
`KeysNativeRedisAcceptanceTest` to use a cluster due to the slots issue?
   
   I had previously thought that `scan` and `keys` didn't apply to clusters, 
but I was wrong, so those are added now. Thanks!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to