----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53092/#review153572 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/cache/client/internal/EndpointManagerImpl.java (line 74) <https://reviews.apache.org/r/53092/#comment222966> this code finds that it currently has no endpoint (endpoint == null) or the one it had has been closed (endpoint.isClosed). In both those cases you have it call listener.clearPdxRegistry which then calls PdxRegistryRecoveryListener.endPointNowInUse. The does not make logical sense to me. The endpoint that is now in use would be the one created on line 76, the new endpoint you are adding. The PdxRegistryRecoveryListener does not even use the "endpoint" argument but I wonder if you should be doing this on line 77 with the new endpoint. - Darrel Schneider On Oct. 21, 2016, 10:57 a.m., Hitesh Khamesra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53092/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2016, 10:57 a.m.) > > > Review request for geode, Bruce Schuchardt, Udo Kohlmeyer, and Dan Smith. > > > Repository: geode > > > Description > ------- > > We clear pdx registry in client when we re-connect to cluster. But during > that time other thread may end up using old registry while other thread is > clearing this. Thus we need to synchronize that. In current code it happens > through listener events and I modified that notification to synchronize this. > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/cache/client/internal/EndpointManagerImpl.java > 3f3d725 > > Diff: https://reviews.apache.org/r/53092/diff/ > > > Testing > ------- > > > Thanks, > > Hitesh Khamesra > >