Skip to content

Comments

KAFKA-16334: Remove --bootstrap-servers from kafka-streams-application-reset#16437

Merged
mjsax merged 1 commit intoapache:trunkfrom
caioguedes:fix/KAFKA-16334
Aug 23, 2024
Merged

KAFKA-16334: Remove --bootstrap-servers from kafka-streams-application-reset#16437
mjsax merged 1 commit intoapache:trunkfrom
caioguedes:fix/KAFKA-16334

Conversation

@caioguedes
Copy link
Contributor

Remove deprecated option --bootstrap-servers from StreamsResetter. Only --bootstrap-server (without s) will be supported.

@caioguedes caioguedes changed the title fix: Remove --bootstrap-servers from kafka-streams-application-reset KAFKA-16334: Remove --bootstrap-servers from kafka-streams-application-reset Jun 24, 2024
@chia7712
Copy link
Member

@caioguedes thanks for this patch. however, we can't merge it as next release is 3.9

@caioguedes
Copy link
Contributor Author

caioguedes commented Jun 25, 2024

Thanks @chia7712!
Yeah I realized that, seems that the changes are planned for 4.0.0 actually. So we'll have to wait...

I'm just curious about the CI, it failed but I couldn't spot the error, could you help me to figure out?

@chia7712
Copy link
Member

I'm just curious about the CI, it failed but I couldn't spot the error, could you help me to figure out?

our tests are not all stable ... so that is expected result :)

@mjsax mjsax merged commit c7f0ade into apache:trunk Aug 23, 2024
@mjsax
Copy link
Member

mjsax commented Aug 23, 2024

Thanks for the PR. Merged to trunk.

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.

3 participants