-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support kafka 0.10.x.x #405
Conversation
Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! 😄 |
@fuji-151a Nice work. Mind agreeing to the terms pointed out above so this PR can be merged? |
@bigdata4u Thank, you!! |
@yahoocla could you merge this PR |
@fuji-151a please add tests for in ClusterConfig serialization/deserialization |
@fuji-151a have you tested it with version 0.10.2.1? |
Thanks for the review @patelh !!
Yes, I have tested it with kafka version 0.10.2.1. |
lgtm |
Thanks guys |
I've added feature that was be able to select version 0.10.1.1, 0.10.2.0 and 0.10.2.1 from UI.
Tests are passing and the instance I'm running locally seems to be working properly.
And He is saying too. (See: #394)
Please check this pull request.
Thank you.