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.

Sign up for GitHub

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

Jump to bottom

MINOR: Update dependencies for Kafka 2.5 #7909

Merged
merged 3 commits into from Jan 9, 2020
Merged

MINOR: Update dependencies for Kafka 2.5 #7909

merged 3 commits into from Jan 9, 2020

Conversation

ijuma
Copy link
Contributor

@ijuma ijuma commented Jan 8, 2020 •
edited

Noteworthy:

Note that I did not upgrade lz4 due to lz4/lz4-java#156.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

aljoshare and others added 2 commits January 8, 2020 06:25
@aljoshare @ijuma
Fix KAFKA-8786
5192dd4
@ijuma
MINOR: Update dependencies for Kafka 2.5
1d15b15 
* Jackson
* ZSTD Performance improvements
* EasyMock/PowerMock
* Gradle plugins
* Reflections no longer depends on Guava
...
@ijuma ijuma requested a review from omkreddy January 8, 2020 14:26
@ijuma ijuma mentioned this pull request Jan 8, 2020
Merged
3 tasks
omkreddy
omkreddy approved these changes Jan 8, 2020
View reviewed changes
Copy link
Contributor

@omkreddy omkreddy left a comment

Choose a reason for hiding this comment

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

@ijuma Thanks for the PR. LGTM.

@ijuma
Copy link
Contributor Author

ijuma commented Jan 8, 2020

retest this please

@ijuma
Copy link
Contributor Author

ijuma commented Jan 9, 2020

1 job passed and the other had a single flaky failure:

kafka.api.PlaintextProducerSendTest.testNonBlockingProducer

@ijuma ijuma merged commit a024e67 into apache:trunk Jan 9, 2020
@ijuma
Copy link
Contributor Author

ijuma commented Jan 9, 2020

Merged to trunk.

@ijuma ijuma mentioned this pull request Jan 9, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@omkreddy omkreddy omkreddy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@ijuma @omkreddy @aljoshare