You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "arnabnandy7 (via GitHub)" <gi...@apache.org> on 2026/09/21 17:04:48 UTC
[PR] KAFKA-19028: Preserve alter config policy compatibility [kafka]
arnabnandy7 opened a new pull request, #23545:
URL: https://github.com/apache/kafka/pull/23545
## Summary
Restore the established `AlterConfigPolicy.RequestMetadata.configs()` behavior for incremental DELETE operations. A DELETE of an unset topic configuration is omitted from the policy map because it generates no config record. A broker DELETE remains present with a `null` value, including when the broker configuration is unset.
This preserves the KRaft behavior that the ZooKeeper compatibility flag introduced by KIP-1252 was designed to match. The Javadoc now describes the record-based behavior. Exposing deleted keys independently would require a separate public API proposal.
## Testing
Updated the focused controller unit test to assert the policy arguments for an unset topic DELETE, a stored topic DELETE, and an unset broker DELETE. The test and Gradle checks have not been run locally. Recommended verification:
```text
./gradlew :metadata:test --tests org.apache.kafka.controller.ConfigurationControlManagerTest.testIncrementalDeletePolicyPreservesNoOpTopicBehavior
./gradlew spotlessCheck
```
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KAFKA-19028: Expose deleted configs to AlterConfigPolicy without changing configs() [kafka]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #23545:
URL: https://github.com/apache/kafka/pull/23545#issuecomment-5883005003
A label of 'needs-attention' was automatically added to this PR in order to raise the
attention of the committers. Once this issue has been triaged, the `triage` label
should be removed to prevent this automation from happening again.
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KAFKA-19028: Expose deleted configs to AlterConfigPolicy without changing configs() [kafka]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #23545:
URL: https://github.com/apache/kafka/pull/23545#issuecomment-5924110105
A label of 'needs-attention' was automatically added to this PR in order to raise the
attention of the committers. Once this issue has been triaged, the `triage` label
should be removed to prevent this automation from happening again.
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] KAFKA-19028: Preserve alter config policy compatibility [kafka]
Posted by "arnabnandy7 (via GitHub)" <gi...@apache.org>.
arnabnandy7 commented on PR #23545:
URL: https://github.com/apache/kafka/pull/23545#issuecomment-5764500867
@chia7712 please review
cc @edoardocomar
--
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: jira-unsubscribe@kafka.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org