You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "Bill (GitHub)" <gi...@apache.org> on 2019/11/15 19:18:34 UTC

[GitHub] [geode] Bill commented on pull request #4313: GEODE-7456: Moving GMSMembershipManager into gms package

I understand the intention here is to ensure that nothing in `geode-core` uses `org.apache.geode.distributed.internal.membership.gms.api.Membership` directly—that within `geode-core` all membership functionality is delivered by `MembershipManagerAdapter` (sic). If that's right, it might be a good idea to add that ArchUnit test to this PR.

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