You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "coderzc (via GitHub)" <gi...@apache.org> on 2026/05/15 07:19:04 UTC
[PR] [fix][broker] Inherit broker offload extra configs from empty policies [pulsar]
coderzc opened a new pull request, #25781:
URL: https://github.com/apache/pulsar/pull/25781
### Motivation
#25736 made broker-level `managedLedgerOffloadExtraConfig*` entries available to `OffloadPoliciesImpl.mergeConfiguration(...)`. However, `managedLedgerExtraConfigurations` defaults to an empty map, so a namespace or topic offload policy with no extra config still stops fallback to broker-level extra config.
This can produce merged offload policies that inherit the broker driver/bucket/region but lose broker-level extra config such as a tiered-storage bucket prefix.
### Modifications
- Merge `managedLedgerExtraConfigurations` as a map from broker -> namespace -> topic.
- Treat null or empty higher-level extra config maps as unset so broker-level entries are inherited.
- Preserve higher-level key overrides, including an explicit empty-string value to clear a broker-level key.
- Add regression coverage for empty namespace-level extra config inheriting broker-level extra config.
### Verifying this change
- `./gradlew :pulsar-common:test --tests org.apache.pulsar.common.policies.data.OffloadPoliciesTest`
--
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
Re: [PR] [fix][broker] Use effective offload policies for extra configs [pulsar]
Posted by "coderzc (via GitHub)" <gi...@apache.org>.
coderzc merged PR #25781:
URL: https://github.com/apache/pulsar/pull/25781
--
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