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




geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java
 (line 953)
<https://reviews.apache.org/r/44587/#comment184905>

    This seems like a hack to have the product code look for a test class???
    
    It would be better to have the product code have a callback that can be 
installed by the test code. Some forceDisconnectListener maybe?


- Dan Smith


On March 9, 2016, 7:27 p.m., Bruce Schuchardt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/44587/
> -----------------------------------------------------------
> 
> (Updated March 9, 2016, 7:27 p.m.)
> 
> 
> Review request for geode, Hitesh Khamesra, Jianxia Chen, and Udo Kohlmeyer.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> We're seeing a number of similar failures that all seem to be caused by JVMs 
> pausing and being kicked out of the distributed system.   This change-set 
> enables creation of a heap dump if a member is forced out of the system and 
> JVM pauses have been detected.  This will give us artifacts that we can 
> analyze to help determine what's going on.
> 
> 
> Diffs
> -----
> 
>   
> geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java
>  a19369942ba691eef8d021fd7d8a537939cf2cc8 
> 
> Diff: https://reviews.apache.org/r/44587/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Bruce Schuchardt
> 
>

Reply via email to