Skip to content

Conversation

@iwasakims
Copy link
Member

No description provided.

@iwasakims
Copy link
Member Author

This will conflict with the patch of BIGTOP-3285 since this includes the change to the version of Kafka. I will rebase this once BIGTOP-3285 is done.

@iwasakims
Copy link
Member Author

I rebased this since BIGTOP-3285 was committed. The unit tests of flume-kafka-source are passed with the patch even kafka.version is 2.4.0 (instead of 2.3.1) on my local.

@sekikn
Copy link
Contributor

sekikn commented Mar 15, 2020

+1, successfully built on CentOS 7 & 8, Debian 9 & 10 and Ubuntu 16.04 & 18.04. It also passed the smoke test.

@sekikn sekikn merged commit 85218c9 into apache:master Mar 15, 2020
@iwasakims
Copy link
Member Author

Thanks, @sekikn!

evans-ye pushed a commit to evans-ye/bigtop that referenced this pull request May 10, 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.

2 participants