Skip to content
New issue

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

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(standalone): require consumer's id to be the same as username #3394

Merged
merged 1 commit into from
Jan 22, 2021

Conversation

spacewander
Copy link
Member

We already did the same thing with the configuration from etcd.

Signed-off-by: spacewander spacewanderlzx@gmail.com

What this PR does / why we need it:

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

We already did the same thing with the configuration from etcd.

Signed-off-by: spacewander <spacewanderlzx@gmail.com>
@spacewander spacewander marked this pull request as ready for review January 22, 2021 03:24
@Firstsawyou
Copy link
Contributor

LGTM.

@membphis membphis merged commit c8d35cd into apache:master Jan 22, 2021
@spacewander spacewander mentioned this pull request Mar 24, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants