----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45132/#review124689 -----------------------------------------------------------
geode-core/src/main/resources/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/jgroups-config.xml (line 23) <https://reviews.apache.org/r/45132/#comment187313> How about just call it MEMBERSHIP_PORT_RANGE? - Jianxia Chen On March 21, 2016, 11:25 p.m., Bruce Schuchardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45132/ > ----------------------------------------------------------- > > (Updated March 21, 2016, 11:25 p.m.) > > > Review request for geode, Hitesh Khamesra, Jianxia Chen, and Udo Kohlmeyer. > > > Bugs: GEODE-1115 > https://issues.apache.org/jira/browse/GEODE-1115 > > > Repository: geode > > > Description > ------- > > added 1 more port to the jgroups port_range and renamed the place-holder > string in jgroups configuration files > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessenger.java > 9476debc727ce389aa699aade32864905d9825c2 > > geode-core/src/main/resources/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/jgroups-config.xml > 841336436c288f3cac4f8a5e43320fa158cd3662 > > geode-core/src/main/resources/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/jgroups-mcast.xml > ff5a8e132304f8b247867df17af9ddf5acda9b4a > > geode-core/src/test/java/com/gemstone/gemfire/distributed/DistributedSystemDUnitTest.java > f34fd7ecd94d75bcbeed71c9049a4688a56abff7 > > Diff: https://reviews.apache.org/r/45132/diff/ > > > Testing > ------- > > added unit test ensuring jgroups is properly configured > > > Thanks, > > Bruce Schuchardt > >
