----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60312/#review179009 -----------------------------------------------------------
The diff doesn't apply cleanly on develop on my machine. What's the base commit? geode-assembly/build.gradle Lines 143 (patched) <https://reviews.apache.org/r/60312/#comment253398> It's a nit, but it would be nice to remove trailing whitespace. - Galen O'Sullivan On June 21, 2017, 9:35 p.m., Hitesh Khamesra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60312/ > ----------------------------------------------------------- > > (Updated June 21, 2017, 9:35 p.m.) > > > Review request for geode, Alexander Murmann, Bruce Schuchardt, and Galen > O'Sullivan. > > > Repository: geode > > > Description > ------- > > GEODE-2804 Cache InetAddress if configure host as ip string. > > 1. We keep configure host string in HostAddress class > 2. We reuse InetsocketAddress if it is ipString. > 3. Client has logic to retry thus we keep InetsocketAddress even if > it is not ip string. > > GEODE-3017 IPv6 issue on windows. Above changes fixed this issue. > > GEODE-2940 Now we don't validate configure host string on start. As > there is possibility that host may not available. > > 1. Earlier wan config were failing because of that. Now we just keep > those configure host string. And try this later. > > Added unit tests for it. > > > Diffs > ----- > > geode-assembly/build.gradle 39bb542 > geode-assembly/src/test/resources/expected_jars.txt 6260167 > geode-core/build.gradle 7f34b4a > > geode-core/src/main/java/org/apache/geode/admin/internal/DistributionLocatorImpl.java > c1bfc93 > > geode-core/src/main/java/org/apache/geode/cache/client/internal/AutoConnectionSourceImpl.java > 070451c > > geode-core/src/main/java/org/apache/geode/cache/client/internal/PoolImpl.java > 3ded54a > > geode-core/src/main/java/org/apache/geode/distributed/internal/AbstractDistributionConfig.java > 01c6157 > > geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java > 1572355 > > geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/GMSUtil.java > c6bef57 > > geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/locator/GMSLocator.java > 93fa9da > > geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/membership/GMSJoinLeave.java > 84d42cf > > geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/membership/HostAddress.java > PRE-CREATION > > geode-core/src/main/java/org/apache/geode/distributed/internal/tcpserver/TcpClient.java > e9476b5 > > geode-core/src/main/java/org/apache/geode/internal/admin/remote/DistributionLocatorId.java > 5ab1bed > > geode-core/src/main/java/org/apache/geode/internal/admin/remote/RemoteTransportConfig.java > 1dc2fd1 > > geode-core/src/main/java/org/apache/geode/internal/cache/ClusterConfigurationLoader.java > 4f4881f > > geode-core/src/main/java/org/apache/geode/internal/cache/PoolFactoryImpl.java > d4fdbd0 > > geode-core/src/test/java/org/apache/geode/cache/client/internal/AutoConnectionSourceDUnitTest.java > 789d326 > > geode-core/src/test/java/org/apache/geode/cache/client/internal/AutoConnectionSourceImplJUnitTest.java > 9169904 > > geode-core/src/test/java/org/apache/geode/distributed/internal/DistributionConfigJUnitTest.java > 9f6c5fb > > geode-core/src/test/java/org/apache/geode/distributed/internal/StartupMessageDataJUnitTest.java > 9d63050 > > geode-core/src/test/java/org/apache/geode/distributed/internal/membership/gms/membership/GMSJoinLeaveJUnitTest.java > a31fa8d > > geode-core/src/test/resources/org/apache/geode/codeAnalysis/excludedClasses.txt > 6a6e0c1 > > geode-wan/src/main/java/org/apache/geode/cache/client/internal/locator/wan/LocatorDiscovery.java > f5a8fcf > > geode-wan/src/main/java/org/apache/geode/cache/client/internal/locator/wan/LocatorMembershipListenerImpl.java > d6d5d7c > > geode-wan/src/main/java/org/apache/geode/internal/cache/wan/AbstractRemoteGatewaySender.java > dbc2cc6 > > geode-wan/src/test/java/org/apache/geode/internal/cache/wan/misc/WanAutoDiscoveryDUnitTest.java > 6d75064 > gradle/dependency-versions.properties 183dafc > > > Diff: https://reviews.apache.org/r/60312/diff/1/ > > > Testing > ------- > > > Thanks, > > Hitesh Khamesra > >
