pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard

[fix][broker] Fix the applying of namespace policies

Open nodece opened this issue 1 year ago • 3 comments

Motivation

I'm using the rate limiter on the replicator, when an existing topic is initializing, and the replicator's cursor exists, the topic will create the replicator, and then load and apply the namespace and topic policies to the topic(which includes the replicator), but I found the rate limiter doesn't work.

The namespace policies will not be applied to the replicator after the broker restart.

When will this happen:

  • The topic policies are disabled.
  • The topic policies are enabled, and there are no policies.

Modifications

  • Loading the compaction service and replicator after the namespace and topic policies are loaded
  • When there are no topic policies, apply the namespace policies to the topic
  • Add resourceGroup, encryptionRequired, and allowAutoUpdateSchema to the HierarchyTopicPolicies to unify the policies

Documentation

  • [ ] doc
  • [ ] doc-required
  • [x] doc-not-needed
  • [ ] doc-complete

nodece avatar Apr 15 '24 05:04 nodece

Check CI errors first

liudezhi2098 avatar Apr 16 '24 01:04 liudezhi2098

/pulsarbot rerun-failure-checks

nodece avatar Apr 16 '24 03:04 nodece

@liudezhi2098 The CI has been passed.

nodece avatar Apr 16 '24 13:04 nodece

Closed by https://github.com/apache/pulsar/pull/22890

nodece avatar Jun 28 '24 06:06 nodece