Skip to content

[improve][broker] PIP-307 Added assignedBrokerUrl to CloseProducerCmd to skip lookups upon producer reconnections during unloading - #21516

Closed
heesung-sohn wants to merge 1 commit into
apache:masterfrom
heesung-sohn:pip-307-commit-fix
Closed

[improve][broker] PIP-307 Added assignedBrokerUrl to CloseProducerCmd to skip lookups upon producer reconnections during unloading#21516
heesung-sohn wants to merge 1 commit into
apache:masterfrom
heesung-sohn:pip-307-commit-fix

Conversation

@heesung-sohn

Copy link
Copy Markdown
Contributor

Motivation

Squash the preivous PR, #21408

Modifications

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

… to skip lookups upon producer reconnections during unloading
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Nov 6, 2023
@heesung-sohn
heesung-sohn deleted the pip-307-commit-fix branch November 6, 2023 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant