Skip to content

KAFKA-12986: Validate throttled replica entries before parsing - #23559

Open
AkshathaTR13 wants to merge 4 commits into
apache:trunkfrom
AkshathaTR13:kafka-12986-validate-throttled-replicas
Open

AkshathaTR13 wants to merge 4 commits into
apache:trunkfrom
AkshathaTR13:kafka-12986-validate-throttled-replicas

Conversation

@AkshathaTR13

Copy link
Copy Markdown

Summary

  • Reject leading and internal empty entries in throttled replica lists while preserving reset values and trailing commas.
  • Reject partition or broker IDs that cannot be parsed as Java integers.
  • Trim individual entries consistently in the runtime parser.
  • Add regression coverage for the Jira reproducer and related parser/validator mismatches.

Testing

  • git diff --check
  • Direct 22-case validation/parsing matrix: all cases passed.
  • The targeted Gradle test could not start in the local workspace because Gradle 9.7.1 could not be downloaded; GitHub CI will run the repository tests.

Generated-by: OpenAI Codex

Reject invalid replica list entries and integer overflow.

Generated-by: OpenAI Codex
Keep runtime parsing consistent with configuration validation.\n\nGenerated-by: OpenAI Codex
Cover empty entries, wildcard combinations, and integer overflow.\n\nGenerated-by: OpenAI Codex
Verify parsing remains consistent with accepted whitespace.\n\nGenerated-by: OpenAI Codex
@github-actions github-actions Bot added triage PRs from the community core Kafka Broker storage Pull requests that target the storage module small Small PRs labels Sep 23, 2026
@AkshathaTR13

Copy link
Copy Markdown
Author

Hi Kafka maintainers, this is my first Apache Kafka code contribution. When convenient, could someone please approve the CI workflow and review this fix? Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Kafka Broker small Small PRs storage Pull requests that target the storage module triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant