BalaKaza commented on a change in pull request #7513:
URL: https://github.com/apache/geode/pull/7513#discussion_r839976234
##########
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:
@jdeppe-pivotal and I looked through the object references for
`valueReference` and `newValueReference` in the case of local references and
they are not the same. I might have introduced a false positive by not adding a
null check in there which I am adding now. With null checks added the test
would fail with moving it to a local reference.
--
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]
