[GitHub] [geode] alb3rtobr commented on issue #4978: Fix for regression introduced by GEODE-7565

[GitBox <gi...@apache.org>]

alb3rtobr commented on issue #4978:
URL: https://github.com/apache/geode/pull/4978#issuecomment-617861442


   > One additional comment regarding the following waning message:
   > 
   > ```
   > [warn 2020/04/18 23:44:22.757 PDT <ServerConnection on port 29019 Thread 1> tid=0x298] Unable to ping non-member rs-FullRegression19040559a2i32xlarge-hydra-client-63(bridgegemfire1_host1_4749:4749)<ec><v39>:41003 for client identity(rs-FullRegression19040559a2i32xlarge-hydra-client-63(edgegemfire3_host1_1071:1071:loner):50046:5a182991:edgegemfire3_host1_1071,connection=2
   > ```
   > 
   > The above is logged by member `bridgegemfire1_host1_4749` in the tests, exactly the member to which the `ping` command was sent... this warning is logged within the `pingCorrectServer` method which, unless I'm missing something, shouldn't be invoked at all if this member is the actual recipient for the `ping` message. Maybe `!myID.equals(targetServer)` is not working as expected?.
   
   I have added a log right after the comparision to check which values were considered different. I have created two clusters with two servers, and in my case I dont see the comparision is not working, this is an example:
   ```
   [info 2020/04/22 15:05:01.338 GMT <ServerConnection on port 32000 Thread 12> tid=0x56] ALBERTO - MyID=172.17.0.3(server-0:85)<v1>:41000 - target=172.17.0.9(server-1:84)<v2>:41000
   ```
   Could you add the same log to your code?


----------------------------------------------------------------
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:
users@infra.apache.org