Skip to content

[cleanup][broker]Cleanup check replication clusters is null#16117

Closed
AnonHxy wants to merge 1 commit intoapache:masterfrom
AnonHxy:cleanup_checkReplication
Closed

[cleanup][broker]Cleanup check replication clusters is null#16117
AnonHxy wants to merge 1 commit intoapache:masterfrom
AnonHxy:cleanup_checkReplication

Conversation

@AnonHxy
Copy link
Contributor

@AnonHxy AnonHxy commented Jun 18, 2022

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

  • Make sure that the change passes the CI checks.

Need to update docs?

  • doc-not-needed
    (Please explain why)

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jun 18, 2022
@AnonHxy AnonHxy closed this Jun 18, 2022
@AnonHxy AnonHxy deleted the cleanup_checkReplication branch June 18, 2022 04:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant