----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53093/#review153582 -----------------------------------------------------------
Ship it! Ship It! - Kevin Duling On Oct. 21, 2016, 11:20 a.m., Jinmei Liao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53093/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2016, 11:20 a.m.) > > > Review request for geode, Kevin Duling, Kirk Lund, and Swapnil Bawaskar. > > > Repository: geode > > > Description > ------- > > GEODE-2025: do not use 8080 as the default http-server-port since we defined > default in DistributionConfig > > When we pulled in a pull request in Auguest, we didn't notice that the > default http-service-port it's using is 8080. In DistributionConfig, we > defined it to be 7070. This would cause a discrepency between the default > http port we are using between locators and servers. change this to 7070 now. > > > Diffs > ----- > > > geode-assembly/src/test/java/org/apache/geode/rest/internal/web/GeodeRestClient.java > c83cebb745ff931e28b81781a53952baf9465bd1 > > geode-assembly/src/test/java/org/apache/geode/rest/internal/web/RestSecurityIntegrationTest.java > 5f66f3bd48055cd0b65726710d36c4fac1d30b40 > > geode-assembly/src/test/java/org/apache/geode/rest/internal/web/RestServersJUnitTest.java > PRE-CREATION > geode-core/src/main/java/org/apache/geode/cache/server/CacheServer.java > 910f34d448a27497a541793d8a6c260e56970d00 > > geode-core/src/main/java/org/apache/geode/internal/cache/execute/util/FindRestEnabledServersFunction.java > 793f73c8cae62188815cf29fb98da8148cb140a8 > > geode-core/src/main/java/org/apache/geode/management/internal/RestAgent.java > 6e7ba7ddd9c21532c1c4978b6af81012a482b782 > > geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/LauncherLifecycleCommands.java > 892a92d688a88de5742e70f28ff5442358a3b218 > > Diff: https://reviews.apache.org/r/53093/diff/ > > > Testing > ------- > > > Thanks, > > Jinmei Liao > >