Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[issue-373] Changed list consumer groups fetching way from all to valid #374

Merged
merged 1 commit into from Jun 19, 2022

Conversation

occidere
Copy link
Contributor

Related Issues


Contents

  • As I wrote in issue above, everytime I access to topic page, ClusterAuthorizationException occurred
  • So, I changed list consumer groups fetching policy from all() to valid() to ensure see topic page without consumer at least

PR Points

  • I tested it with my cluster that occurred this error and It's working well
    • Before change (used all())
      topic page error
    • After change (used valid())
      fixed topic page
  • I think it is not a critical change but If you have any unclear things or any questions, just tell me anytime 馃檹

Copy link
Collaborator

@davideicardi davideicardi left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@davideicardi davideicardi merged commit ef97d8d into obsidiandynamics:master Jun 19, 2022
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.

None yet

2 participants