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

Review request for geode, Bruce Schuchardt, Hitesh Khamesra, Jared Stewart, 
Kevin Duling, and Udo Kohlmeyer.


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


Repository: geode


Description
-------

GEODE-2297: push new InternalDistributedSystem into AlertAppender

* InternalDistributedSystem initialize pushes its instance into
AlertAppender

* During AlertAppender shuttingDown, it nulls out the reference
to InternalDistributedSystem

This is an initial fix to prevent deadlocks. After merging this fix in, I'd 
like to start refactoring AlertAppender as a follow-up.


Diffs
-----

  
geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java
 fad9206583fdc093b1f74397c321442335cf80d4 
  
geode-core/src/main/java/org/apache/geode/internal/logging/log4j/AlertAppender.java
 a6c54aba352ba4dd3ece94c94f770292db6e9284 

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


Testing
-------

precheckin in progress


Thanks,

Kirk Lund

Reply via email to