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

[ZEPPELIN-6118] Remove unnecessary hadoop3 profile #4861

Merged
merged 1 commit into from
Oct 5, 2024

Conversation

pan3793
Copy link
Member

@pan3793 pan3793 commented Oct 3, 2024

What is this PR for?

Given that hadoop2 support was removed, hadoop3 is the only option, and it's unlikely that hadoop4 has existed in predictable years. We can remove such a profile to simplify the pom.xml files.

What type of PR is it?

Improvement

Todos

  • remove profile hadoop3 and update CI workflow yamls and docs
  • rename profile hadoop3-aws to hadoop-aws
  • rename profile hadoop3-azure to hadoop-azure
  • given hadoop deps are defined at root pom.xml's dependencyManagement block, simplify sub-projects' pom.xml by removing redundant attributes.

What is the Jira issue?

ZEPPELIN-6118

How should this be tested?

Pass GHA.

Screenshots (if appropriate)

Questions:

  • Does the license files need to update? No.
  • Is there breaking changes for older versions? No.
  • Does this needs documentation? No.

@pan3793 pan3793 requested review from Reamer and jongyoul October 4, 2024 06:08
@pan3793 pan3793 merged commit ed2ee4d into apache:master Oct 5, 2024
27 of 28 checks passed
pan3793 added a commit that referenced this pull request Oct 5, 2024
### What is this PR for?

Given that hadoop2 support was removed, hadoop3 is the only option, and it's unlikely that hadoop4 has existed in predictable years. We can remove such a profile to simplify the `pom.xml` files.

### What type of PR is it?

Improvement

### Todos

- [x] remove profile `hadoop3` and update CI workflow yamls and docs
- [x] rename profile `hadoop3-aws` to `hadoop-aws`
- [x] rename profile `hadoop3-azure` to `hadoop-azure`
- [x] given hadoop deps are defined at root `pom.xml`'s `dependencyManagement` block, simplify sub-projects' `pom.xml` by removing redundant attributes.

### What is the Jira issue?

ZEPPELIN-6118

### How should this be tested?

Pass GHA.
### Screenshots (if appropriate)

### Questions:
* Does the license files need to update? No.
* Is there breaking changes for older versions? No.
* Does this needs documentation? No.

Closes #4861 from pan3793/ZEPPELIN-6118.

Signed-off-by: Cheng Pan <[email protected]>
(cherry picked from commit ed2ee4d)
Signed-off-by: Cheng Pan <[email protected]>
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