You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@geode.apache.org by Jinmei Liao <ji...@pivotal.io> on 2016/10/21 18:19:51 UTC

Review Request 53093: GEODE-2025: do not use 8080 as the default http-server-port since we defined default in DistributionConfig

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53093/
-----------------------------------------------------------

Review request for geode.


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