You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by "kirklund (GitHub)" <gi...@apache.org> on 2019/01/11 21:17:48 UTC

[GitHub] [geode] kirklund opened pull request #3071: GEODE-6232: Extract ResourceEventNotifier from InternalDistributedSystem

This PR does not fix GEODE-6232 but is a necessary step in delivering the fix.

Please review @demery-pivotal @mhansonp 

[ Full content available at: https://github.com/apache/geode/pull/3071 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] kirklund closed pull request #3071: GEODE-6232: Extract ResourceEventNotifier from InternalDistributedSystem

Posted by "kirklund (GitHub)" <gi...@apache.org>.
[ pull request closed by kirklund ]

[ Full content available at: https://github.com/apache/geode/pull/3071 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] kirklund commented on issue #3071: GEODE-6232: Extract ResourceEventNotifier from InternalDistributedSystem

Posted by "kirklund (GitHub)" <gi...@apache.org>.
This PR needs to wait for PR #3087 to be merged in.

[ Full content available at: https://github.com/apache/geode/pull/3071 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] kirklund commented on issue #3071: GEODE-6232: Extract ResourceEventNotifier from InternalDistributedSystem

Posted by "kirklund (GitHub)" <gi...@apache.org>.
@galen-pivotal At this point of continued test failures with no clear cause, I create a new branch and repeat the refactoring in smaller steps that each go through precheckin. I'm going to close the PR and plan to go in that direction.

[ Full content available at: https://github.com/apache/geode/pull/3071 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] kirklund commented on issue #3071: GEODE-6232: Extract ResourceEventNotifier from InternalDistributedSystem

Posted by "kirklund (GitHub)" <gi...@apache.org>.
This branch seems to reintroduce GEODE-6285 in some way.

[ Full content available at: https://github.com/apache/geode/pull/3071 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org