Skip to content

KAFKA-18068: Fix the typo PARTITIONER_ADPATIVE_PARTITIONING_ENABLE in ProducerConfig#20317

Merged
frankvicky merged 1 commit into
apache:trunkfrom
mingyen066:KAFKA-18068-Fix-the-typo-`PARTITIONER_ADPATIVE_PARTITIONING_ENABLE`-in-ProducerConfig
Aug 7, 2025
Merged

KAFKA-18068: Fix the typo PARTITIONER_ADPATIVE_PARTITIONING_ENABLE in ProducerConfig#20317
frankvicky merged 1 commit into
apache:trunkfrom
mingyen066:KAFKA-18068-Fix-the-typo-`PARTITIONER_ADPATIVE_PARTITIONING_ENABLE`-in-ProducerConfig

Conversation

@mingyen066

@mingyen066 mingyen066 commented Aug 6, 2025

Copy link
Copy Markdown
Collaborator

Fixes a typo in ProducerConfig: Renames
PARTITIONER_ADPATIVE_PARTITIONING_ENABLE_CONFIG
PARTITIONER_ADAPTIVE_PARTITIONING_ENABLE_CONFIG

The old key is retained for backward compatibility.

See: KIP-1175: Fix the typo PARTITIONER_ADPATIVE_PARTITIONING_ENABLE
in ProducerConfig

Reviewers: Yung yungyung7654321@gmail.com, TengYao Chi
frankvicky@apache.org, Ken Huang s7133700@gmail.com, Nick Guo
lansg0504@gmail.com, Ranuga Disansa go2ranuga@gmail.com

@github-actions github-actions Bot added triage PRs from the community producer clients small Small PRs labels Aug 6, 2025

@Yunyung Yunyung left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions Bot removed the triage PRs from the community label Aug 7, 2025

@frankvicky frankvicky left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mingyen066: LGTM

@frankvicky frankvicky merged commit 2329def into apache:trunk Aug 7, 2025
31 checks passed
/** <code>partitioner.adaptive.partitioning.enable</code> */
public static final String PARTITIONER_ADPATIVE_PARTITIONING_ENABLE_CONFIG = "partitioner.adaptive.partitioning.enable";
private static final String PARTITIONER_ADPATIVE_PARTITIONING_ENABLE_DOC =
public static final String PARTITIONER_ADAPTIVE_PARTITIONING_ENABLE_CONFIG = "partitioner.adaptive.partitioning.enable";

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a minor deprecation, but it doesn't hurt to include it in upgrade.html, right?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot to update upgrade.html.
#20322 will fix that.

chia7712 pushed a commit that referenced this pull request Aug 7, 2025
…_ENABLE_CONFIG (#20322)

Document deprecation of PARTITIONER_ADPATIVE_PARTITIONING_ENABLE_CONFIG
in `upgrade.html`, which was missed in #20317

Reviewers: Ken Huang <s7133700@gmail.com>, Chia-Ping Tsai
 <chia7712@gmail.com>
@mingyen066 mingyen066 deleted the KAFKA-18068-Fix-the-typo-`PARTITIONER_ADPATIVE_PARTITIONING_ENABLE`-in-ProducerConfig branch February 13, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants