geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ken Howe <kh...@pivotal.io>
Subject Review Request 61701: GEODE-3277: Fix error path constructors of Launcher inner State classess
Date Wed, 16 Aug 2017 21:21:57 GMT

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

Review request for geode, Emily Yeh, Jinmei Liao, Jared Stewart, Kirk Lund, and Patrick Rhomberg.


Repository: geode


Description
-------

Updated tests for changes in the error constructors for ServerState and
LocatorState.

Minor spelling corrections.

I've updated the fix for GEODE-3277 and rebased on top of the @klund's recent process controller
updates


Diffs
-----

  geode-assembly/src/test/java/org/apache/geode/management/internal/cli/commands/StatusLocatorRealGfshTest.java
3a98373938e3de21da6badcf460dae3648218ac6 
  geode-core/src/main/java/org/apache/geode/distributed/LocatorLauncher.java 83c1ab533e3dea323a8a99f7002b9464a54dfc25

  geode-core/src/main/java/org/apache/geode/distributed/ServerLauncher.java ae64691605130c9b212a3a33bb65ae37b28af02b

  geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/lifecycle/GfshStatusCommandsIntegrationTest.java
dd5841f4cffca38da07a11f381cf4174d7264349 
  geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
e7f17ef208a1708f385c7c4041affb70fd309a4c 


Diff: https://reviews.apache.org/r/61701/diff/1/


Testing
-------

Precheckin is in progress.


Thanks,

Ken Howe


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message