pivotal-eshu commented on a change in pull request #6054:
URL: https://github.com/apache/geode/pull/6054#discussion_r586866920
##########
File path:
geode-core/src/distributedTest/java/org/apache/geode/internal/cache/ClientServerTransactionDUnitTest.java
##########
@@ -3782,13 +3790,15 @@ public Object call() throws Exception {
@Override
public Object call() throws Exception {
TXManagerImpl mgr = getGemfireCache().getTxManager();
+ mgr.resume(txid);
Review comment:
After you resume the transaction, the following code trying to resume
can be safely removed.
##########
File path:
geode-core/src/distributedTest/java/org/apache/geode/internal/cache/ClientServerTransactionDUnitTest.java
##########
@@ -3749,14 +3756,15 @@ public Object call() throws Exception {
lr.validatedPut(event, System.currentTimeMillis());
lr.validatedPut(event, System.currentTimeMillis());
lr.validatedPut(event, System.currentTimeMillis());
- return null;
+ return mgr.suspend();
}
});
server.invoke(new SerializableCallable() {
@Override
public Object call() throws Exception {
TXManagerImpl mgr = getGemfireCache().getTxManager();
+ // mgr.resume(txid2);
Review comment:
You may want to clean up this commented out line.
----------------------------------------------------------------
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]
