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