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 2021/02/04 07:07:58 UTC

[GitHub] [pulsar] yangl opened a new pull request #9475: [pulsar-broker]Cleanup all metadata inited by initialize-cluster-meta…

yangl opened a new pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475


   Fixes #9474
   


----------------------------------------------------------------
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



[GitHub] [pulsar] yangl closed pull request #9475: [pulsar-broker] fix the delete-cluster-metadata command parameter "-cluster"

Posted by GitBox <gi...@apache.org>.
yangl closed pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475


   


----------------------------------------------------------------
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



[GitHub] [pulsar] BewareMyPower commented on pull request #9475: [pulsar-broker] fix the delete-cluster-metadata command parameter "-cluster"

Posted by GitBox <gi...@apache.org>.
BewareMyPower commented on pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475#issuecomment-773388173


   First please follow https://github.com/apache/pulsar/blob/master/.github/PULL_REQUEST_TEMPLATE.md to edit your PR description.
   
   For the code issue, there's already a `ClusterMetadataTearDownTest` under package `org.apache.pulsar.tests.integration.cli`.  Your new test seems to be covered by the test and it only checks a few z-nodes.


----------------------------------------------------------------
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



[GitHub] [pulsar] BewareMyPower commented on pull request #9475: [pulsar-broker] fix the delete-cluster-metadata command parameter "-cluster"

Posted by GitBox <gi...@apache.org>.
BewareMyPower commented on pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475#issuecomment-773388173


   First please follow https://github.com/apache/pulsar/blob/master/.github/PULL_REQUEST_TEMPLATE.md to edit your PR description.
   
   For the code issue, there's already a `ClusterMetadataTearDownTest` under package `org.apache.pulsar.tests.integration.cli`.  Your new test seems to be covered by the test and it only checks a few z-nodes.


----------------------------------------------------------------
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



[GitHub] [pulsar] yangl commented on pull request #9475: [pulsar-broker]Cleanup all metadata inited by initialize-cluster-meta…

Posted by GitBox <gi...@apache.org>.
yangl commented on pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475#issuecomment-773128180


   after the init, the zk path as follow:
   the --configuration-store zk contains **`admin pulsar`**
   the --zookeeper  zk  contains  **`namespace stream bookies managed-ledgers`**
   the --existing-bk-metadata-service-uri zk contains  **`/chroot/ledgers`**


----------------------------------------------------------------
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



[GitHub] [pulsar] yangl removed a comment on pull request #9475: [pulsar-broker]Cleanup all metadata inited by initialize-cluster-meta…

Posted by GitBox <gi...@apache.org>.
yangl removed a comment on pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475#issuecomment-773128180


   after the init, the zk path as follow:
   the --configuration-store zk contains **`admin pulsar`**
   the --zookeeper  zk  contains  **`namespace stream bookies managed-ledgers`**
   the --existing-bk-metadata-service-uri zk contains  **`/chroot/ledgers`**


----------------------------------------------------------------
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



[GitHub] [pulsar] yangl closed pull request #9475: [pulsar-broker] fix the delete-cluster-metadata command parameter "-cluster"

Posted by GitBox <gi...@apache.org>.
yangl closed pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475


   


----------------------------------------------------------------
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



[GitHub] [pulsar] yangl commented on pull request #9475: [pulsar-broker]Cleanup all metadata inited by initialize-cluster-meta…

Posted by GitBox <gi...@apache.org>.
yangl commented on pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475#issuecomment-773128180


   after the init, the zk path as follow:
   the --configuration-store zk contains **`admin pulsar`**
   the --zookeeper  zk  contains  **`namespace stream bookies managed-ledgers`**
   the --existing-bk-metadata-service-uri zk contains  **`/chroot/ledgers`**


----------------------------------------------------------------
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



[GitHub] [pulsar] yangl removed a comment on pull request #9475: [pulsar-broker]Cleanup all metadata inited by initialize-cluster-meta…

Posted by GitBox <gi...@apache.org>.
yangl removed a comment on pull request #9475:
URL: https://github.com/apache/pulsar/pull/9475#issuecomment-773128180


   after the init, the zk path as follow:
   the --configuration-store zk contains **`admin pulsar`**
   the --zookeeper  zk  contains  **`namespace stream bookies managed-ledgers`**
   the --existing-bk-metadata-service-uri zk contains  **`/chroot/ledgers`**


----------------------------------------------------------------
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