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/06/18 03:45:41 UTC

[GitHub] [pulsar] AnonHxy opened a new pull request, #16117: [cleanup][broker]Cleanup check replication clusters is null

AnonHxy opened a new pull request, #16117:
URL: https://github.com/apache/pulsar/pull/16117

   ### Motivation
   
   
   * There is no need to  check `policies.getReplicationClusters() != null` because we have used `HierarchyTopicPolicies#replicationClusters`
   
   ### Modifications
   
   * Remove  checking `policies.getReplicationClusters() != null`
   
   ### Verifying this change
   
   - [x] Make sure that the change passes the CI checks.
   
   
   Need to update docs? 
   
   - [x] `doc-not-needed` 
   (Please explain why)


-- 
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] AnonHxy closed pull request #16117: [cleanup][broker]Cleanup check replication clusters is null

Posted by GitBox <gi...@apache.org>.

AnonHxy closed pull request #16117: [cleanup][broker]Cleanup check replication clusters is null
URL: https://github.com/apache/pulsar/pull/16117


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