You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2019/06/06 15:12:09 UTC
[GitHub] [pulsar] marutronix removed a comment on issue #4471:
[pulsar-broker] add cli/admin api to delete bookie-affinity group
marutronix removed a comment on issue #4471: [pulsar-broker] add cli/admin api to delete bookie-affinity group
URL: https://github.com/apache/pulsar/pull/4471#issuecomment-499422252
FUCK
2019年6月5日(水) 3:07 Rajan Dhabalia <no...@github.com>:
> Motivation
>
> This PR is on top of #4458 <https://github.com/apache/pulsar/pull/4458>.
> It adds support to delete bookie-affinity group using cli/admin api.
> ------------------------------
> You can view, comment on, or merge this pull request online at:
>
> https://github.com/apache/pulsar/pull/4471
> Commit Summary
>
> - [pulsar-broker] support secondary bookie isolation group at namespace
> - [pulsar-broker] add cli/admin api to delete bookie-affinity group
>
> File Changes
>
> - *M*
> pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-0> (17)
> - *M*
> pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/Namespaces.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-1> (21)
> - *M*
> pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-2> (21)
> - *M*
> pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-3> (6)
> - *M*
> pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BrokerBookieIsolationTest.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-4> (238)
> - *M*
> pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/Namespaces.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-5> (31)
> - *M*
> pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/internal/NamespacesImpl.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-6> (22)
> - *M*
> pulsar-client-tools-test/src/test/java/org/apache/pulsar/admin/cli/PulsarAdminToolTest.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-7> (10)
> - *M*
> pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdNamespaces.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-8> (28)
> - *A*
> pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/BookieAffinityGroupData.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-9> (56)
> - *M*
> pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/LocalPolicies.java
> <https://github.com/apache/pulsar/pull/4471/files#diff-10> (6)
>
> Patch Links:
>
> - https://github.com/apache/pulsar/pull/4471.patch
> - https://github.com/apache/pulsar/pull/4471.diff
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/pulsar/pull/4471?email_source=notifications&email_token=ALK4OTBD2RQ2SBTRMZE4PA3PY2VP5A5CNFSM4HS7UEJ2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GXTA4VA>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ALK4OTFPWPRG4UQ73QDZ7PLPY2VP5ANCNFSM4HS7UEJQ>
> .
>
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services