----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60312/#review178995 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/cache/client/internal/AutoConnectionSourceImpl.java Line 86 (original), 89 (patched) <https://reviews.apache.org/r/60312/#comment253371> Since you're poking around in here anyways, can you clean up the formatting here geode-core/src/main/java/org/apache/geode/cache/client/internal/AutoConnectionSourceImpl.java Line 106 (original), 109 (patched) <https://reviews.apache.org/r/60312/#comment253373> This doesn't appear to be the most useful javadoc ever, but at the very least you should add the new parameter to it. geode-core/src/main/java/org/apache/geode/cache/client/internal/AutoConnectionSourceImpl.java Lines 117 (patched) <https://reviews.apache.org/r/60312/#comment253375> Did you mean to leave this comment in? geode-core/src/main/java/org/apache/geode/distributed/internal/tcpserver/TcpClient.java Line 161 (original), 161 (patched) <https://reviews.apache.org/r/60312/#comment253391> Looks like you wanted to ask Bruce about this, you probably shouldn't merge this comment. geode-core/src/main/java/org/apache/geode/internal/admin/remote/DistributionLocatorId.java Lines 235 (patched) <https://reviews.apache.org/r/60312/#comment253396> Can we remove all of the the this.<variable> calls in favor of just using <variable> directly? Failing that, can you change 225 and 226 to this.hostname for consistency. geode-core/src/main/java/org/apache/geode/internal/cache/PoolFactoryImpl.java Lines 59 (patched) <https://reviews.apache.org/r/60312/#comment253397> Typo, s/loators/locators geode-core/src/main/java/org/apache/geode/internal/cache/PoolFactoryImpl.java Lines 308 (patched) <https://reviews.apache.org/r/60312/#comment253399> remove this - Brian Rowe 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 > >