Skip to content

MINOR: Kafka Streams should log broker provided KIP-1071 config values#22730

Open
mjsax wants to merge 1 commit into
apache:trunkfrom
mjsax:minor-1071-log-broker-provided-configs
Open

MINOR: Kafka Streams should log broker provided KIP-1071 config values#22730
mjsax wants to merge 1 commit into
apache:trunkfrom
mjsax:minor-1071-log-broker-provided-configs

Conversation

@mjsax

@mjsax mjsax commented Jul 2, 2026

Copy link
Copy Markdown
Member

In the "streams" protocol, certain configs moved from the client to the
broker, and the broker provides some of these configs to the client via
the heartbeat response. The client should log these values for better
client-side observability.

In the "streams" protocol, certain configs moved from the client to the broker,
and the broker provides some of these configs to the client via the heartbeat
response. The client should log these values for better client-side
observability.
@mjsax mjsax added streams kip Requires or implements a KIP labels Jul 2, 2026
}

@Test
public void testTaskOffsetsReportedForStandbyAndWarmupTasks() {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This test is not directly related to this PR, but closes an existing test gap -- I just added it to avoid opening an extra PR. Hope that's ok. :)

}

@Test
public void testTaskOffsetReportingCadenceConditionsInterplay() {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Same. Just closing an existing testing gap.

}

@Test
public void testStreamsRebalanceDataTaskOffsetIntervalMsUpdatedOnSuccess() {

@mjsax mjsax Jul 2, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

And two more :)

@mjsax mjsax changed the title MINOR: Kafka Streams should log broker provided KIP-1017 config values MINOR: Kafka Streams should log broker provided KIP-1071 config values Jul 2, 2026
@mjsax mjsax added KIP-1071 PRs related to KIP-1071 and removed kip Requires or implements a KIP labels Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant