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:42 UTC

[geode] branch feature/GEODE-4144 created (now 148192f)

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

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


      at 148192f  GEODE-4144: EventId in client does not match that of server (with putAll)

This branch includes the following new commits:

     new 148192f  GEODE-4144: EventId in client does not match that of server (with putAll)

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


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

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

Posted by la...@apache.org.
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>.