----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39738/ -----------------------------------------------------------
Review request for geode and Jianxia Chen. Repository: geode Description ------- We had quite a bit of convoluted code in GMSJoinLeave after implementing the tcp/ip failure detection check. This change-set moves the location of failure-detection port information out of GMSHealthMonitor and into the NetView class so that it is always carried with the view. This decouples the health monitor and join-leave and keeps the port information and member information together in a common store. This also allows us to remove the propagation of request messages from many ViewCreator methods and remove the port-lookup code from join-leave. Diffs ----- gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/NetView.java cb61a1ba6e90939f026ddffc1983d274b50e3e15 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/fd/GMSHealthMonitor.java d89ba39265478583a407f4803a40d08302c55af3 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/interfaces/HealthMonitor.java 628e416a3718bb02b9657d716017e7b0e8658368 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java 5633424b69bb5b35242015da1d557af358b2f13d gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messages/InstallViewMessage.java 1afdf401ce93c03af1da929f0cd6712c152d0211 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messages/JoinRequestMessage.java 952b20e3fda830f86729d60c7f20513d69dae7eb gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messages/JoinResponseMessage.java 77b72c348be93c1cda623694b3336725748ceefe gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessenger.java ae7ee16b818b09a7315210e5c427486ce841ab60 gemfire-core/src/main/java/com/gemstone/gemfire/internal/util/PluckStacks.java 15a64d7441c09e178eaef40598f32226a69c8d00 gemfire-core/src/test/java/com/gemstone/gemfire/cache30/ReconnectDUnitTest.java 26e85869f920fdc6044c059709012e3a9056dfab gemfire-core/src/test/java/com/gemstone/gemfire/distributed/LocatorDUnitTest.java 8779c6d7fe66ef55cf293c19f5f66b7a3ed20823 gemfire-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeaveJUnitTest.java 4e3293221e59d45cc8b2e9486b9e008b34225231 gemfire-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessengerJUnitTest.java 2b9d451a5c24fbff740c97658e1e58eef34f5dc8 Diff: https://reviews.apache.org/r/39738/diff/ Testing ------- unit testing (underway) Thanks, Bruce Schuchardt
