jdeppe-pivotal commented on a change in pull request #5937:
URL: https://github.com/apache/geode/pull/5937#discussion_r561977373
##########
File path:
geode-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/server/AbstractHitsMissesIntegrationTest.java
##########
@@ -244,28 +392,24 @@ public void testSunionstore() {
runDiffStoreCommandAndAssertNoStatUpdates("set", (k, v, s) ->
jedis.sunionstore(k, v, s));
}
- // ------------ Hash related commands -----------
-
+ // TODO our implementation updates the stats when it shouldn't
+ @Ignore
Review comment:
I've opened GEODE-8857 to fix SMOVE. Please add this Jira to the
`@Ignore` here.
##########
File path:
geode-redis/src/acceptanceTest/java/org/apache/geode/redis/internal/executor/server/InfoNativeRedisAcceptanceTest.java
##########
@@ -28,9 +28,4 @@
public int getPort() {
return redis.getPort();
}
-
- @Override
- int getExposedPort() {
Review comment:
You will probably need to revert this. When run under docker, there are
2 ports in play - one is the external port exposed for outside access and one
is the internal port which the external port is proxied to. It's the internal
port that this is returning and that's the port that the Info command will
report.
----------------------------------------------------------------
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]
