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 2022/05/16 08:06:11 UTC

[GitHub] [pulsar] Technoboy- opened a new pull request, #15619: [PIP][broker] PIP-162: Enable system topic by default.

Technoboy- opened a new pull request, #15619:
URL: https://github.com/apache/pulsar/pull/15619

   Fixes #15333
   
   ### Motivation
   In [PIP-39](https://github.com/apache/pulsar/wiki/PIP-39%3A-Namespace-Change-Events), we have introduced namespace event to support topic level policy. This feature was released in Pulsar 2.6.
   
   Then [PIP-92](https://github.com/apache/pulsar/wiki/PIP-92%3A-Topic-policy-across-multiple-clusters), we support topic policy to replicate across clusters and release in Pulsar 2.10.
   
   And with version iteration, the corresponding topic policy CLI and functions are mature and stable, but users need to enable system topic to support these features.  So it's better to enable system topic by default.
   
   ### Documentation
   
     
   - [x] `doc` 
   (Your PR contains doc changes)
   


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

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] Technoboy- commented on pull request #15619: [PIP][broker] PIP-162: Enable system topic by default.

Posted by GitBox <gi...@apache.org>.
Technoboy- commented on PR #15619:
URL: https://github.com/apache/pulsar/pull/15619#issuecomment-1130879263

   /pulsarbot run-failure-checks


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

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] Technoboy- commented on pull request #15619: [PIP][broker]PIP-162: Enable system topic by default.

Posted by GitBox <gi...@apache.org>.
Technoboy- commented on PR #15619:
URL: https://github.com/apache/pulsar/pull/15619#issuecomment-1134090976

   > Great work! I found some tests must disable the system topic and topic level policies, I'm not sure if there are break changes or if the test requires disabling topic level policies.
   
   Because the old behavior is disable the system topic. so we need to disable it explicitly so that some test could pass. Otherwise like metrics maybe affect by the system topic.


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

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] Technoboy- merged pull request #15619: [PIP][broker]PIP-162: Enable system topic by default.

Posted by GitBox <gi...@apache.org>.
Technoboy- merged PR #15619:
URL: https://github.com/apache/pulsar/pull/15619


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

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org