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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update fork #1

Merged
merged 15 commits into from
Sep 17, 2020
Merged

Update fork #1

merged 15 commits into from
Sep 17, 2020

Conversation

swenzel
Copy link

@swenzel swenzel commented Sep 17, 2020

No description provided.

Tincu Gabriel and others added 15 commits May 5, 2020 14:29
* Add logic for inferring newer broker versions

- New Fetch / ListOffsets request / response objects
- Add new test cases to inferr code based on mentioned objects
- Add unit test to check inferred version against whatever resides in KAFKA_VERSION
- Add new kafka broker versions to integration list
- Add more kafka broker versions to travis task list
- Add support for broker version 2.5 id

* Implement PR change requests: fewer versions for travis testing, remove unused older versions for inference code, remove one minor version from known server list
Do not use newly created ACL request / responses in allowed version lists, due to flexible versions enabling in kafka actually requiring a serialization protocol header update
Revert admin client file change
Co-authored-by: MostafaElmenabawy <momenabawy@synapse-analytics.io>
Co-authored-by: MostafaElmenabawy <momenabawy@synapse-analytics.io>
Adding namedtuples for DescribeConsumerGroup response; Adding Serialization of MemberData and MemberAssignment for the response

Co-authored-by: Apurva Telang <atelang@paypal.com>
Co-authored-by: Jeff Widman <jeff@jeffwidman.com>
Currently there's no way to pass timeout to check_version if called from admin.
Fix initialization order in KafkaClient
@swenzel swenzel merged commit f31b57c into real-digital:master Sep 17, 2020
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.