You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by la...@apache.org on 2018/01/04 02:06:43 UTC

[geode] 01/01: GEODE-4144: EventId in client does not match that of server (with putAll)

This is an automated email from the ASF dual-hosted git repository.

ladyvader pushed a commit to branch feature/GEODE-4144
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 148192f618593e5c8999a1ff6ff67142ccae4d60
Author: Lynn Hughes-Godfrey <lh...@pivotal.io>
AuthorDate: Wed Jan 3 18:04:57 2018 -0800

    GEODE-4144: EventId in client does not match that of server (with putAll)
    
    - Added better logging of EventId (including MembershipID) when this failure is encountered.
---
 .../java/org/apache/geode/internal/cache/ha/PutAllDUnitTest.java     | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/geode-core/src/test/java/org/apache/geode/internal/cache/ha/PutAllDUnitTest.java b/geode-core/src/test/java/org/apache/geode/internal/cache/ha/PutAllDUnitTest.java
index 4cb3bb1..1c9e972 100755
--- a/geode-core/src/test/java/org/apache/geode/internal/cache/ha/PutAllDUnitTest.java
+++ b/geode-core/src/test/java/org/apache/geode/internal/cache/ha/PutAllDUnitTest.java
@@ -369,7 +369,10 @@ public class PutAllDUnitTest extends JUnit4DistributedTestCase {
     for (int i = 0; i < 5; i++) {
       assertNotNull(eventIds1[i]);
       assertNotNull(eventIds2[i]);
-      assertEquals(eventIds1[i], eventIds2[i]);
+      assertEquals(
+          "Event id mismatch: eventIds1[" + i + "]" + eventIds1[i].expensiveToString()
+              + ": eventIds2[" + i + "]" + eventIds2[i].expensiveToString(),
+          eventIds1[i], eventIds2[i]);
     }
     for (int i = 0; i < 5; i++) {
       assertNotNull(eventIds1[i]);

-- 
To stop receiving notification emails like this one, please contact
"commits@geode.apache.org" <co...@geode.apache.org>.