coderzc opened a new pull request, #25736: URL: https://github.com/apache/pulsar/pull/25736
### Motivation `OffloadPoliciesImpl.create(Properties)` recognizes broker config entries with the `managedLedgerOffloadExtraConfig` prefix and stores them in `managedLedgerExtraConfigurations`. However, `OffloadPoliciesImpl.mergeConfiguration(...)` did not apply the same parsing when broker properties were used as the fallback layer. As a result, applied offload policies could omit broker-level extra offloader configuration such as `managedLedgerOffloadExtraConfigtieredStorageBucketPrefix`. ### Modifications - Reuse a helper to parse `managedLedgerOffloadExtraConfig*` entries from `Properties`. - Apply that parsing in `mergeConfiguration(...)` for `managedLedgerExtraConfigurations`. - Add tests for broker-level extra config merge and higher-level overrides, including empty-string override. ### 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
