----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44940/#review123955 -----------------------------------------------------------
geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java (line 1458) <https://reviews.apache.org/r/44940/#comment186277> Does it require "msg.setSender(this.localAddress)" before sending the message? - Jianxia Chen On March 16, 2016, 10:57 p.m., Hitesh Khamesra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44940/ > ----------------------------------------------------------- > > (Updated March 16, 2016, 10:57 p.m.) > > > Review request for geode, Bruce Schuchardt, Jianxia Chen, and Udo Kohlmeyer. > > > Repository: geode > > > Description > ------- > > there is possibility that existing member can see view from non-existing > memebr in view. Thus now we just ignore that view, if member is not exist > in current view. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java > 2b7893e > > Diff: https://reviews.apache.org/r/44940/diff/ > > > Testing > ------- > > > Thanks, > > Hitesh Khamesra > >
