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

(Updated June 20, 2015, 6:10 a.m.)


Review request for geode, Darrel Schneider and Kirk Lund.


Bugs: GEODE-51
    https://issues.apache.org/jira/browse/GEODE-51


Repository: geode


Description
-------

hostname-for-clients property was being ignored by server startup, preventing 
external clients to connect to the cache servers after connecting to the 
locator.


Diffs (updated)
-----

  
gemfire-core/src/main/java/com/gemstone/gemfire/distributed/ServerLauncher.java 
d5db59d 
  
gemfire-core/src/main/java/com/gemstone/gemfire/distributed/ServerLauncher.java 
d5db59d 
  
gemfire-core/src/main/java/com/gemstone/gemfire/internal/i18n/LocalizedStrings.java
 d5ea8ab 
  
gemfire-core/src/main/java/com/gemstone/gemfire/management/internal/cli/commands/LauncherLifecycleCommands.java
 68b2483 
  
gemfire-core/src/test/java/com/gemstone/gemfire/distributed/ServerLauncherJUnitTest.java
 5af33e1 

Diff: https://reviews.apache.org/r/35542/diff/


Testing
-------

Passed all JUnit tests.
Manual verification using Docker-composer example.
JMX MBean values are OK.


File Attachments
----------------

GEODE-51.patch v2
  
https://reviews.apache.org/media/uploaded/files/2015/06/20/01916fb1-9f7d-4847-8a10-59a9ac0824c5__GEODE-51.patch


Thanks,

William Markito

Reply via email to