[jira] [Commented] (GEODE-3820) CI failure: DataSerializerPropogationDUnitTest.testDataSerializersEventIdVerification

["Shelley Lynn Hughes-Godfrey (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GEODE-3820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16489779#comment-16489779 ] 

Shelley Lynn Hughes-Godfrey commented on GEODE-3820:
----------------------------------------------------

Reopened as this failure reproduced on geode 1.7 branch:
https://concourse.apachegeode-ci.info/teams/main/pipelines/release-1.7.0/jobs/DistributedTest/builds/3

{noformat}
org.apache.geode.internal.cache.tier.sockets.DataSerializerPropagationDUnitTest > testDataSerializersEventIdVerificationClientsAndServers FAILED
    java.lang.AssertionError: EventId found Different
        at org.junit.Assert.fail(Assert.java:88)
        at org.junit.Assert.assertTrue(Assert.java:41)
        at org.apache.geode.internal.cache.tier.sockets.DataSerializerPropagationDUnitTest.testDataSerializersEventIdVerificationClientsAndServers(DataSerializerPropagationDUnitTest.java:408)
{noformat}

Comments from Barry:
I know Jason and I looked at a bug awhile ago (GEODE-4144) with event ids being different. It was a timing issue with the port being set, but I'm not sure if this could affect this.

If I were debugging this, I would add some logging in the afterReceivingFromServer callback if the event ids were different here:
{noformat}
private static void setClientServerObserver2() {
    PoolImpl.IS_INSTANTIATOR_CALLBACK = true;
    ClientServerObserverHolder.setInstance(new ClientServerObserverAdapter() {
        @Override
        public void afterReceivingFromServer(EventID eventID) {
            testEventIDResult = eventID.equals(eventId);
        }
    });
}
{noformat}

> CI failure: DataSerializerPropogationDUnitTest.testDataSerializersEventIdVerification
> -------------------------------------------------------------------------------------
>
>                 Key: GEODE-3820
>                 URL: https://issues.apache.org/jira/browse/GEODE-3820
>             Project: Geode
>          Issue Type: Bug
>          Components: serialization
>    Affects Versions: 1.7.0
>            Reporter: Bruce Schuchardt
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.6.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> This test contains many pauses & needs to be converted to use Awaitility.
> The spelling of this class name should also be corrected.
> org.apache.geode.internal.cache.tier.sockets.DataSerializerPropogationDUnitTest > testDataSerializersEventIdVerificationClientsAndServers FAILED
>     java.lang.AssertionError: EventId found Different
>         at org.junit.Assert.fail(Assert.java:88)
>         at org.junit.Assert.assertTrue(Assert.java:41)
>         at org.apache.geode.internal.cache.tier.sockets.DataSerializerPropogationDUnitTest.testDataSerializersEventIdVerificationClientsAndServers(DataSerializerPropogationDUnitTest.java:673)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)