[GitHub] [geode] pivotal-eshu commented on pull request #2571: GEODE-5828: fixes replicates miss transaction commit.

2018-10-05 Thread GitHub
Will address this concern with additional dunit test. [ Full content available at: https://github.com/apache/geode/pull/2571 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] pivotal-eshu commented on pull request #2571: GEODE-5828: fixes replicates miss transaction commit.

2018-10-05 Thread GitHub
txTracker.commitProcessReceived method no longer need distribution manager anymore. [ Full content available at: https://github.com/apache/geode/pull/2571 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] pivotal-eshu commented on pull request #2571: GEODE-5828: fixes replicates miss transaction commit.

2018-10-04 Thread GitHub
You are right that it is no longer in inner class. Cleaned up the code. [ Full content available at: https://github.com/apache/geode/pull/2571 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org