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-5969] Remove Hadoop2 Support #4674

Closed
wants to merge 12 commits into from
Closed

Conversation

Reamer
Copy link
Contributor

@Reamer Reamer commented Oct 11, 2023

What is this PR for?

This pull request removes support for Hadoop 2.

What type of PR is it?

Improvement

Todos

  • - Fix CI
  • - Check License files

What is the Jira issue?

How should this be tested?

  • CI

Questions:

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

@pan3793
Copy link
Member

pan3793 commented Oct 12, 2023

Removing Hadoop2 support would be a big change, I'm not very familiar with Flink, but for Spark, it requires at least Spark 3.x to support Hadoop 3, the Livy module still uses Spark 2.4.8.

And Spark moved to use Hadoop Shaded client since 3.2.0, which is a quite big change for Hadoop deps management.

I suppose it requires much effort to complete this task, as the community is preparing for the 0.11 release, better to postpone this change to the next release.

</properties>
</profile>
<profile>
<id>hive1</id>
Copy link
Member

Choose a reason for hiding this comment

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

I think it requires a dedicated JIRA ticket to remove Hive 1.x on Flink interpreter

@@ -409,7 +409,7 @@

<plugin>
<artifactId>maven-surefire-plugin</artifactId>
<configuration combine.children="append">
Copy link
Member

Choose a reason for hiding this comment

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

why change this?

@@ -80,9 +80,6 @@ public static String downloadFlink(String flinkVersion, String scalaVersion) {
runShellCommand(new String[]{"wget",
"https://repo1.maven.org/maven2/org/apache/hive/hive-exec/2.3.4/hive-exec-2.3.4.jar",
"-P", targetFlinkHomeFolder + "/lib"});
runShellCommand(new String[]{"wget",
"https://repo1.maven.org/maven2/org/apache/flink/flink-shaded-hadoop2-uber/2.7.5-1.8.1/flink-shaded-hadoop2-uber-2.7.5-1.8.1.jar",
Copy link
Member

Choose a reason for hiding this comment

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

@Reamer
Copy link
Contributor Author

Reamer commented Apr 2, 2024

Closed in favour of #4691

@Reamer Reamer closed this Apr 2, 2024
@Reamer Reamer deleted the hadoop2 branch April 2, 2024 08:26
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