dschneider-pivotal commented on a change in pull request #6814:
URL: https://github.com/apache/geode/pull/6814#discussion_r703803122
##########
File path:
geode-apis-compatible-with-redis/src/integrationTest/java/org/apache/geode/redis/internal/pubsub/SubscriptionsIntegrationTest.java
##########
@@ -53,49 +54,53 @@
};
}
+ final Subscriptions subscriptions = new Subscriptions();
+
+ private int dummyCount;
Review comment:
It gets reset for each test method because a new instance of the test
class is created.
But given this test is doing concurrency testing I am changing this counter
to an AtomicInteger to make sure two concurrent threads do not get the same
dummyCount value.
--
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]
