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

BIGTOP-3721: Add Kafka support for Bigtop 3.1.0 Mpack #930

Merged
merged 3 commits into from Jul 8, 2022

Conversation

kevinw66
Copy link
Contributor

@kevinw66 kevinw66 commented Jul 5, 2022

Description of PR

Add Kafka support for Bigtop 3.1.0 Mpack

Tested in CentOS7

How was this patch tested?

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
  • Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

@guyuqi
Copy link
Member

guyuqi commented Jul 5, 2022

@kevinw66 Thanks for working on it.
Pls give me some time to try to deploy this Kafa service by Mpack.

@kevinw66 kevinw66 changed the title BIGTOP-3689: Add Kafka support for Bigtop 3.1.0 Mpack BIGTOP-3721: Add Kafka support for Bigtop 3.1.0 Mpack Jul 6, 2022
@guyuqi
Copy link
Member

guyuqi commented Jul 6, 2022

Integrated Mpacks PRs (#934 #933 #932 #931) with this one, Hadoop, ZK and Kafka can be successfully deployed by Mpack with Bigtop-3.1.0:

kafka1
KAFKA2

+1, LGTM

I'd like to merge this PR after more tests for (#934 #933 #932 #931) .

@kevinw66
Copy link
Contributor Author

kevinw66 commented Jul 7, 2022

Seems like there are problem when we Restart a running Kafka Broker.
But when click Stop then Start, it works fine.
Let me check on this.

@guyuqi
Copy link
Member

guyuqi commented Jul 8, 2022

Ran stop, start, restart Kafka services and both succeeded.
+1

@guyuqi guyuqi merged commit 8b9628a into apache:master Jul 8, 2022
@guyuqi
Copy link
Member

guyuqi commented Jul 8, 2022

Merged it. Thanks, @kevinw66

PRs: #930 #931 #932 #933 #934 make Mpack support Bigtop-3.1 stack.
Verified that the deployment from Ambari-2.7.5+BGTP-Mpack+Bigtop-3.1 on Centos-7 works fine.

The potential issues on other Distros would be tracked by the specific Jira tickets respectively.

@kevinw66 kevinw66 deleted the bigtop-3689 branch July 8, 2022 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants