DonalEvans commented on a change in pull request #7513:
URL: https://github.com/apache/geode/pull/7513#discussion_r839794122



##########
File path: 
geode-for-redis/src/distributedTest/java/org/apache/geode/redis/internal/commands/executor/string/MSetDUnitTest.java
##########
@@ -48,6 +51,8 @@
 
   private static final Logger logger = LogService.getLogger();
 
+  public static RedisString valueReference;

Review comment:
       I'm not sure that I see how changing it to a local variable for the test 
case changes the behaviour of the test. When I make this change, the test still 
passes, and both invocations that use it are on VM 1, so not a different JVM.




-- 
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.

To unsubscribe, e-mail: [email protected]

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


Reply via email to