[ZEPPELIN-6006] Remove command line applications when downloading applications #4746

Reamer · 2024-03-28T09:22:17Z

What is this PR for?

This pull request removes the use of command line applications when downloading Spark, Flink, Hadoop and Livy.
This pull request also updates some Apache Commons libraries, which are primarily required for decompression.

What type of PR is it?

Improvement

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-6006

How should this be tested?

CI

Questions:

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

jongyoul · 2024-03-29T01:32:00Z

I have an idea. In my understanding, build-tools aims to run the check style properly. So I think we'd better make a new module for your improvement, instead of using build-tools. WDYT?

Reamer · 2024-04-02T07:45:09Z

You are right, the build-tools module was set up so that the checkstyle reports work properly and we can distribute the configuration cleanly. (see https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html)

I think it is unnecessary to create another module, so I have moved it in there.

It was important to me that the code is contained in a module that is independent of the Zeppelin interpreter modules and the Zeppelin-zengine/zeppelin server.

I can also put the code in a new module. What do you think?

jongyoul · 2024-04-03T07:58:18Z

First of all, I totally agree with you that deciding the code from zeppelin-interpreter and zengine. BTW, I'm a bit afraid of adding more test-related code later in the module. So even if it's not necessary to add a new module this time, I recommend adding a new module like zeppelin-test and so on for the future. WDYT?

Reamer · 2024-04-03T09:50:27Z

In the long term, you will probably be right. I have stored the code in a new module.

Reamer · 2024-04-09T06:31:26Z

Ready for review.

jongyoul · 2024-04-11T00:56:39Z

zeppelin-test/pom.xml

+      </plugin>
+    </plugins>
+  </build>
+</project>


nit: add a new line at the end of the file.

Thanks for the hint. I often point out this uncleanliness to my colleagues myself. Adjusted.

jongyoul

LGTM except for a tiny comment.

pan3793 · 2024-04-11T02:14:58Z

zeppelin-test/src/main/java/org/apache/zeppelin/test/DownloadUtils.java

+          .setTaskName("Unarchiv")
+          .setUnit("MiB", 1048576) // setting the progress bar to use MiB as the unit
+          .setStyle(ProgressBarStyle.ASCII)
+          .setUpdateIntervalMillis(1000)


we may want to use a large interval in CI console to avoid too many logs. suggest making it configurable via env var

I do not see the use case, but have implemented this via the environment variable PROGRESS_BAR_UPDATE_INTERVAL.

pan3793 · 2024-04-11T02:17:10Z

zeppelin-test/src/main/java/org/apache/zeppelin/test/DownloadUtils.java

+    }
+
+
+    // download other dependencies for running flink with yarn and hive


I touched this part in fa6e3ee, could you please sync the change?

Many thanks for the hint. I have adjusted the download paths.

pan3793 · 2024-04-11T02:19:17Z

zeppelin-test/src/main/java/org/apache/zeppelin/test/DownloadUtils.java

+   * @return return livyHome
+   * @throws IOException
+   */
+  public static String downloadLivy(String livyVersion) {


livy has scala suffix since 0.8, see #4678

as the current master uses 0.7, leave it also fine

I have added methods and related tests to download livy 0.8.0 in the future.

pan3793 · 2024-04-11T02:20:49Z

zeppelin-test/src/test/java/org/apache/zeppelin/test/DownloadUtilsTest.java

+
+  @Test
+  void downloadHadoop() {
+    String hadoopHome = DownloadUtils.downloadHadoop("3.4.0");


seems it has not been tested yet. one of the major changes in 3.4 is switching AWS SDK from v1 to v2, that's kind of a breaking change.

These are only test methods. It is up to the caller to decide which version is finally downloaded.

pan3793 · 2024-04-11T02:27:45Z

zeppelin-test/src/main/java/org/apache/zeppelin/test/DownloadUtils.java

+public class DownloadUtils {
+  private static final Logger LOGGER = LoggerFactory.getLogger(DownloadUtils.class);
+
+  private static final String MIRROR_URL = "https://www.apache.org/dyn/closer.lua?preferred=true";


if possible, please make it configure via env var, i.e. APACHE_MIRROR (Spark uses it in some places, including build/mvn)

Good point. I have adapted it.

pan3793 · 2024-04-11T02:30:51Z

zeppelin-test/src/main/java/org/apache/zeppelin/test/DownloadUtils.java

+          throw new IOException("Failed to create directory " + newFile);
+        }
+      } else {
+        // fix for Windows-created archives


what's the exact issue? do we really support run test on windows?

I have the code from here. I don't think it's out of the question to support other operating systems in the future.
However, there is no comment in their Git. I will therefore also remove it.
https://github.com/eugenp/tutorials/blob/c0559cbb6d6c66c3a87898805e28310a02a52458/core-java-modules/core-java-io/src/main/java/com/baeldung/unzip/UnzipFile.java#L23-L27

pan3793 · 2024-04-11T02:33:02Z

zeppelin-test/src/main/java/org/apache/zeppelin/test/DownloadUtils.java

+   * @param hadoopVersion
+   * @return home of Spark installation
+   */
+  public static String downloadSpark(String sparkVersion, String hadoopVersion) {


maybe we should expose scalaVersion too, Spark 4 is going to release in mid-2024, it only supports Scala 2.13

At the moment I cannot say what the download directory of Spark 4.x looks like. But I have included the Scala version (default null).

pan3793

LGTM.

One additional suggestion, better to upgrade dep in the dedicated patch, some of the version bumping actually fixes CVEs, dedicated patch is easy to backport and track.

jongyoul · 2024-04-12T02:37:35Z

I approved it but I started CI again as it has 8 failed cases including a known one.

By the way, Does anyone have an idea for the known failure?

Reamer · 2024-04-12T05:36:37Z

I will take a look. There is also another failure.

Reamer · 2024-04-12T10:17:04Z

I approved it but I started CI again as it has 8 failed cases including a known one.

By the way, Does anyone have an idea for the known failure?

I have changed a method signature. The tests are now running again.

Reamer · 2024-04-12T10:17:53Z

One additional suggestion, better to upgrade dep in the dedicated patch, some of the version bumping actually fixes CVEs, dedicated patch is easy to backport and track.

I have opened #4757. Who runs the backport?

Reamer · 2024-04-16T07:45:08Z

New approval required.

* [MINOR] Update jdbc.md Change postgres username (apache#4704) change postgres default username mysql_user/mysql_password to pg_user/pg_password. * Bump mathjax from 2.7.0 to 3.0.0 in /zeppelin-web (apache#4705) Bumps [mathjax](https://github.com/mathjax/MathJax) from 2.7.0 to 3.0.0. - [Release notes](https://github.com/mathjax/MathJax/releases) - [Commits](mathjax/MathJax@2.7.0...3.0.0) --- updated-dependencies: - dependency-name: mathjax dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump org.apache.shiro:shiro-core from 1.10.0 to 1.13.0 (apache#4703) Bumps [org.apache.shiro:shiro-core](https://github.com/apache/shiro) from 1.10.0 to 1.13.0. - [Release notes](https://github.com/apache/shiro/releases) - [Changelog](https://github.com/apache/shiro/blob/main/RELEASE-NOTES) - [Commits](apache/shiro@shiro-root-1.10.0...shiro-root-1.13.0) --- updated-dependencies: - dependency-name: org.apache.shiro:shiro-core dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump follow-redirects from 1.15.3 to 1.15.4 in /zeppelin-web (apache#4702) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.3 to 1.15.4. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.3...v1.15.4) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump follow-redirects from 1.15.3 to 1.15.4 in /zeppelin-web-angular (apache#4701) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.3 to 1.15.4. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.3...v1.15.4) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [MINOR] Upgrade jackson version in /zeppelin-server (apache#4635) * [HOTFIX] Disable shell interpreter (apache#4708) * Fix GCSNotebookRepoTests (apache#4711) * [HOTFIX] Check permission when updating cron information (apache#4631) * [HOTFIX] Check permission when updating cron information * [HOTFIX] Fix commented * [HOTFIX] Check permission when updating cron information * [HOTFIX] Check permission when updating cron information * [HOTFIX] Check permission when updating cron information * [MINOR] Change minimum java version to 11 in docs (apache#4710) * [MINOR] Change minimum java version to 11 in docs * [MINOR] Change minimum java version to 11 in docs * [HOTFIX] Change the link of `helium.json` from S3 to zeppelin.apache.org (apache#4713) * [ZEPPELIN-5990] Disable sensitive configuration for JDBC url (apache#4709) * [ZEPPELIN-5990] Disable sensitive configuration for JDBC url * [ZEPPELIN-5990] Disable sensitive configuration for JDBC url * [ZEPPELIN-5995] Update Kubernetes Library and hopefully fix flaky tests (apache#4712) * [MINOR] Set Snapshot version to 0.12.0-SNAPSHOT (apache#4720) * change version to 0.11.1-SNAPSHOT * change version * change to 0.12.0-SNAPSHOT * Bump org.postgresql:postgresql from 42.4.3 to 42.7.2 in /jdbc (apache#4723) Bumps [org.postgresql:postgresql](https://github.com/pgjdbc/pgjdbc) from 42.4.3 to 42.7.2. - [Release notes](https://github.com/pgjdbc/pgjdbc/releases) - [Changelog](https://github.com/pgjdbc/pgjdbc/blob/master/CHANGELOG.md) - [Commits](https://github.com/pgjdbc/pgjdbc/commits) --- updated-dependencies: - dependency-name: org.postgresql:postgresql dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [HOTFIX] Escape Ldap search filters (apache#4714) * [HOTFIX] Escape envs when using `.conf` (apache#4715) * Fix the new zeppelin ui. The specific reason is that the use of excessively wide column widths resulted in the remaining fields being squeezed out of the screen, and nz table did not have a scroll bar set to display scrolling. (apache#4727) * Bump ip from 1.1.8 to 1.1.9 in /zeppelin-web (apache#4724) Bumps [ip](https://github.com/indutny/node-ip) from 1.1.8 to 1.1.9. - [Commits](indutny/node-ip@v1.1.8...v1.1.9) --- updated-dependencies: - dependency-name: ip dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump lodash from 4.17.15 to 4.17.21 in /zeppelin-web-angular (apache#4689) Bumps [lodash](https://github.com/lodash/lodash) from 4.17.15 to 4.17.21. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](lodash/lodash@4.17.15...4.17.21) --- updated-dependencies: - dependency-name: lodash dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [HOTFIX] Escape HeliumPackage information (apache#4728) * [NO-ISSUE] Use reload4j instead of log4j (apache#4719) * [NO-ISSUE] Upgrade org.json jar (apache#4722) * [ZEPPELIN-6001] k8s images fix (apache#4729) * Update zeppelin-interpreter Dockerfile fixed ARG version; Error "bzip2: Compressed file ends unexpectedly;" fixed by adding ---no-iri flag to wget * Update zeppelin-server Dockerfile fixed ARG version; fixed JAVA_HOME env. * Update scripts/docker/zeppelin-interpreter/Dockerfile Co-authored-by: Philipp Dallig <philipp.dallig@gmail.com> --------- Co-authored-by: Philipp Dallig <philipp.dallig@gmail.com> * [ZEPPELIN-6000] Polish some files mainly in zengine (apache#4731) * some misc polish * some misc polish * [ZEPPELIN-6003] Log source info of SQL in JDBCInterpreter (apache#4732) * [ZEPPELIN-6003] Log detail info of SQL in JDBCInterpreter * Update Co-authored-by: Philipp Dallig <philipp.dallig@gmail.com> --------- Co-authored-by: Philipp Dallig <philipp.dallig@gmail.com> * [ZEPPELIN-6002] Fix completer NPE (apache#4730) Co-authored-by: Philipp Dallig <philipp.dallig@gmail.com> * [ZEPPELIN-5986] Bump Maven surefire/failsafe plugins to recover JUnit5 tests (apache#4734) * [ZEPPELIN-5986] Re-enable Junit 5 integration tests by upgrading maven plugins * Fix SparkIntegrationTest * Disable the LivyInterpreterIT * Selenium * NPE * Disable testEditOnDoubleClick * [ZEPPELIN-6005] Update Kyuubi JDBC docs (apache#4738) * Bump org.apache.commons:commons-configuration2 from 2.8.0 to 2.10.1 (apache#4740) Bumps org.apache.commons:commons-configuration2 from 2.8.0 to 2.10.1. --- updated-dependencies: - dependency-name: org.apache.commons:commons-configuration2 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [MINOR] Fix missing changed versions (apache#4737) * [MINOR] Fix missing changed versions * nit * Bump com.nimbusds:nimbus-jose-jwt in /zeppelin-server (apache#4733) Bumps [com.nimbusds:nimbus-jose-jwt](https://bitbucket.org/connect2id/nimbus-jose-jwt) from 9.13 to 9.37.2. - [Changelog](https://bitbucket.org/connect2id/nimbus-jose-jwt/src/master/CHANGELOG.txt) - [Commits](https://bitbucket.org/connect2id/nimbus-jose-jwt/branches/compare/9.37.2..9.13) --- updated-dependencies: - dependency-name: com.nimbusds:nimbus-jose-jwt dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [ZEPPELIN-6007] Enhance release scripts for tar shasum commands detection (apache#4747) * [ZEPPELIN-6007] Enhance release scripts for tar shasum commands detection * fix * nit * indent * [ZEPPELIN-6008] Fix parameter usage of bokeh in test case (apache#4748) * [ZEPPELIN-6008] Pin plotly 5.19.0 * Revert "[ZEPPELIN-6008] Pin plotly 5.19.0" This reverts commit 60d9ce9. * Pin bokeh=3.3.4 * Revert "Pin bokeh=3.3.4" This reverts commit 44ccc98. * Remove usage of deprecated paramater legend * [ZEPPELIN-5969] Remove Hadoop2 and move to Hadoop3 shaded client (apache#4691) * Drop hadoop2 in github actions * Update docs * Drop hadoop2 support * Remove hadoop2 integration tests * findbugs use the same version in all modules * Use hadoop3.3 for tests * Move to scala 2.12 * Try to fix flink * Usage of metals * Remove duplicate version and groupid * Fix Flink with Hadoop3 * fix log * R * fix * fix * fix * fix * hadoop-3.3 * fix * fix * Address comments * address comments --------- Co-authored-by: Philipp Dallig <philipp.dallig@gmail.com> * [HOTFIX] Remove rendering helium description as HTML in Frontend (apache#4755) * Bump express from 4.18.2 to 4.19.2 in /zeppelin-web-angular (apache#4744) Bumps [express](https://github.com/expressjs/express) from 4.18.2 to 4.19.2. - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/master/History.md) - [Commits](expressjs/express@4.18.2...4.19.2) --- updated-dependencies: - dependency-name: express dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * update scripts/docker/zeppelin/bin/Dockerfile to JDK11 (apache#4753) * update scripts/docker/zeppelin/bin/Dockerfile to JDK11 * update miniconda to py3.9 * Update Dockerfile to remove extra line * Bump some apache common libs (apache#4757) * [ZEPPELIN-6006] Remove command line applications when downloading applications (apache#4746) * Move Files with java * Use java to download external dependecies * Improve code after review * Correct Mirror-URL and compilation * [ZEPPELIN-6016] Rewrite and enable Livy integration tests (apache#4743) * wip * nit * nit * wip * wip * fix * [ZEPPELIN-5973] Bump Livy 0.8.0-incubating * nit * Spark 3.5.1 * test * fix * comment * nit * nit * nit * [ZEPPELIN-6017] Revert changes about ZEPPELIN_IDENT_STRING in ZEPPELIN-5421 * [ZEPPELIN-6015] Update ci-action plugins (apache#4759) * [ZEPPELIN-5999] Reduce instance objects from Zeppelin (apache#4726) * Remove ZeppelinConfiguration Singelton and add MiniZeppelinServer * Add ZeppelinConfiguration to Interpreter * Remove static pluginmanager and configstorage * Inject servicelocator into SessionConfiguratior * use custom serviceLocator in integration tests * Reorder code * code cleanup * Add ZeppelinConfiguration as class variable to InterpreterOption * Avoid leaking third-party libs * [ZEPPELIN-6019] Remove Submarine support (apache#4763) * [ZEPPELIN-6022] Skip decryption of credentials.json when file is empty (apache#4765) * Skip decryption when empty * Use more elegant empty json string check * [ZEPPELIN-6018] Update gRPC version from 1.51.0 to 1.55.1 for successful Apache Zeppelin build on s390x architecture (apache#4758) * Update grpc version in pom.xml for successful Apache Zeppelin build on s390x architecture In the s390x architecture, the Apache Zeppelin package builds successfully when the grpc dependency version in the pom.xml file is changed from version 1.51.0 to version 1.62.2. Therefore, I have updated the grpc version in the pom.xml file. * Update LICENSE Updated the grpc version in LICENSE from 1.51.0 to 1.62.2 * grpc-version-change-to 1.55.1 * [ZEPPELIN-6027] Enhanced Integration with Apache Kyuubi (apache#4770) * [ZEPPELIN-6027] Enhanced Integration with Apache Kyuubi * fix style * [ZEPPELIN-6028] Enhance default value assignment for ZEPPELIN_IDENT_STRING (apache#4772) * [ZEPPELIN-6029] Set COPYFILE_DISABLE=1 for macOS tar (apache#4774) * [ZEPPELIN-6029] Add --disable-copyfile for macOS tar * COPYFILE_DISABLE=1 * ODP-1644: Removed unsupported interpreter * ODP-1315: Bumpup loadash to 4.x * ODP: fix odp version * ODP-303 New UI build fix for missing projects * [ODP-1315] ODP-1644: CVE fixes * ODP-1644: Update to Spark Version * ODP-1644: Added libthrift, updated hadoop and phoenix versions * ODP-1644: Hive 4.0.0 support for JDBC * ODP-1644: Added two variables that were previously missed * ODP-1644: Fixed node/npm version issue when testing with arm64 MacOS * ODP-1644: Increased timeouts/sleeps to pass tests * ODP-1644: Increased ms in Thread.sleep * ODP-1829: Updated pom.xml files * ODP-1829: Removed uneeded tests * ODP-1829: Fixed tests failures * ODP-1829: Disabled Finicky Helium Test * ODP-1829: Increased wait time so that tests would not fail * ODP-1829: Thread.Sleep -> Thread.sleep * Zeppelin Build fixes * ODP-1644: Updated version number to 11.2.3.3.6.0-1 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: bigpear0201 <bigpear0201@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Shefali Singh <31477542+shefali163@users.noreply.github.com> Co-authored-by: Jongyoul Lee <jongyoul@gmail.com> Co-authored-by: Philipp Dallig <philipp.dallig@gmail.com> Co-authored-by: PJ Fanning <pjfanning@users.noreply.github.com> Co-authored-by: myongyun <984093369@qq.com> Co-authored-by: th1nksnow <114925852+th1nksnow@users.noreply.github.com> Co-authored-by: Manhua <kevinjmh@qq.com> Co-authored-by: Manhua <manhua@apache.org> Co-authored-by: Cheng Pan <chengpan@apache.org> Co-authored-by: Cheng Pan <pan3793@gmail.com> Co-authored-by: Gayle <9532712+flowy0@users.noreply.github.com> Co-authored-by: zeotuan <48720253+zeotuan@users.noreply.github.com> Co-authored-by: Aditi Sharma <167422409+aditi-sharma-1@users.noreply.github.com> Co-authored-by: Prabhjyot Singh <prabhjyot@acceldata.io> Co-authored-by: shubhamsharma <shubham@acceldata.io>

Reamer force-pushed the downloadutils branch 2 times, most recently from f8fe349 to 27dfd71 Compare March 28, 2024 10:02

Reamer force-pushed the downloadutils branch from 27dfd71 to 8dcd5ab Compare April 2, 2024 07:10

Reamer force-pushed the downloadutils branch from 8dcd5ab to a6f6072 Compare April 3, 2024 07:55

Reamer force-pushed the downloadutils branch 2 times, most recently from 532fe2a to 5e87104 Compare April 3, 2024 09:48

Reamer requested a review from jongyoul April 5, 2024 09:14

jongyoul reviewed Apr 11, 2024

View reviewed changes

jongyoul previously approved these changes Apr 11, 2024

View reviewed changes

pan3793 reviewed Apr 11, 2024

View reviewed changes

Reamer mentioned this pull request Apr 11, 2024

[ZEPPELIN-5999] Reduce instance objects from Zeppelin #4726

Merged

1 task

Reamer dismissed jongyoul’s stale review via e9859ae April 11, 2024 16:20

Reamer force-pushed the downloadutils branch 2 times, most recently from e9859ae to 354f4ee Compare April 11, 2024 16:21

pan3793 approved these changes Apr 11, 2024

View reviewed changes

jongyoul previously approved these changes Apr 12, 2024

View reviewed changes

Reamer dismissed jongyoul’s stale review via 710a0d2 April 12, 2024 08:14

Reamer added 4 commits April 12, 2024 15:54

Move Files with java

4d8ca1f

Use java to download external dependecies

29f1013

Improve code after review

a60f0ef

Correct Mirror-URL and compilation

836c289

Reamer force-pushed the downloadutils branch from 710a0d2 to 836c289 Compare April 12, 2024 13:54

Reamer requested a review from jongyoul April 14, 2024 09:41

jongyoul approved these changes Apr 17, 2024

View reviewed changes

jongyoul merged commit 67098fd into apache:master Apr 17, 2024
27 of 28 checks passed

Reamer deleted the downloadutils branch April 17, 2024 05:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[ZEPPELIN-6006] Remove command line applications when downloading applications #4746

[ZEPPELIN-6006] Remove command line applications when downloading applications #4746

Reamer commented Mar 28, 2024

jongyoul commented Mar 29, 2024

Reamer commented Apr 2, 2024 •

edited

Loading

jongyoul commented Apr 3, 2024

Reamer commented Apr 3, 2024

Reamer commented Apr 9, 2024

jongyoul Apr 11, 2024

Reamer Apr 11, 2024

jongyoul left a comment

pan3793 Apr 11, 2024

Reamer Apr 11, 2024

pan3793 Apr 11, 2024

Reamer Apr 11, 2024

pan3793 Apr 11, 2024

Reamer Apr 11, 2024

pan3793 Apr 11, 2024 •

edited

Loading

Reamer Apr 11, 2024

pan3793 Apr 11, 2024

Reamer Apr 11, 2024

pan3793 Apr 11, 2024

Reamer Apr 11, 2024

pan3793 Apr 11, 2024

Reamer Apr 11, 2024

pan3793 left a comment •

edited

Loading

jongyoul commented Apr 12, 2024

Reamer commented Apr 12, 2024

Reamer commented Apr 12, 2024

Reamer commented Apr 12, 2024

Reamer commented Apr 16, 2024

		}


		// download other dependencies for running flink with yarn and hive

[ZEPPELIN-6006] Remove command line applications when downloading applications #4746

[ZEPPELIN-6006] Remove command line applications when downloading applications #4746

Conversation

Reamer commented Mar 28, 2024

What is this PR for?

What type of PR is it?

What is the Jira issue?

How should this be tested?

Questions:

jongyoul commented Mar 29, 2024

Reamer commented Apr 2, 2024 • edited Loading

jongyoul commented Apr 3, 2024

Reamer commented Apr 3, 2024

Reamer commented Apr 9, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jongyoul left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pan3793 Apr 11, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pan3793 left a comment • edited Loading

Choose a reason for hiding this comment

jongyoul commented Apr 12, 2024

Reamer commented Apr 12, 2024

Reamer commented Apr 12, 2024

Reamer commented Apr 12, 2024

Reamer commented Apr 16, 2024

Reamer commented Apr 2, 2024 •

edited

Loading

pan3793 Apr 11, 2024 •

edited

Loading

pan3793 left a comment •

edited

Loading