nodece commented on PR #23769: URL: https://github.com/apache/pulsar/pull/23769#issuecomment-2559221226
This PR introduces an override for the `replicateSubscriptionState` configuration from the consumer level, which is currently correct for the version being worked on. However, this conflicts slightly with PIP #23770, which proposes a configuration at the namespace and topic levels to control subscription replication. The issue arises because PR #23770 assumes that the consumer-level configuration for replicateSubscriptionState is not set, and instead, replication behavior is driven by namespace and topic policies. To avoid this conflict and align with PIP #23770, I propose modifying this PR to ignore the `replicateSubscriptionState` configuration from the consumer level. This would ensure that subscription replication is controlled exclusively by the namespace and topic policies. If you can set the subscription replication in the namespace or topic level, I belive this should be correct. -- 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]
