Skip to content

[improve][broker] update broker error message to include tenant name - #25503

Merged
merlimat merged 2 commits into
apache:masterfrom
pgier:improve-tenant-error-message
Apr 12, 2026
Merged

[improve][broker] update broker error message to include tenant name#25503
merlimat merged 2 commits into
apache:masterfrom
pgier:improve-tenant-error-message

Conversation

@pgier

@pgier pgier commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Motivation

This is just to improve the log message to include the tenant name when there the tenant is not found

Modifications

Just updated the log

Verifying this change

  • Make sure that the change passes the CI checks.

PR for CI: pgier#13

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

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

No, it does not affect any of these.

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

@dlg99 dlg99 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.

LGTM

@merlimat
merlimat merged commit aec41c4 into apache:master Apr 12, 2026
80 of 82 checks passed
@lhotari lhotari added this to the 5.0.0-M1 milestone Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants