zeppelin icon indicating copy to clipboard operation
zeppelin copied to clipboard

[ZEPPELIN-5812] Add spring-boot-dependencies

Open proceane opened this issue 2 years ago • 4 comments

What is this PR for?

This is the task of adding a spring boot dependency to switch JaxRs and Jersey to Spring.

What type of PR is it?

Improvement

Todos

  • [x] - Add spring-boot-dependencies in zeppelin-server
  • [x] - Upgrade jersey, jetty version
  • Because of conflicts with jersey and jetty versions that exist inside spring-boot-dependencies
  • jersey: 2.35, jetty: 9.4.48.v20220622
  • [x] - Add jackson-core, jackson-annotations, jackson-module-jaxb-annotations dependency
  • Currently, the jackson version set in zeppein-server is 2.12.6.1, but some of the jackson dependencies do not exist this version.
  • jackson-core, jackson-annotations: 2.13.3
  • jackson-module-jaxb-annotations: 2.12.2(Using this version due to version conflict in zeppelin-distribution)

What is the Jira issue?

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

How should this be tested?

CI

Questions:

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

proceane avatar Sep 06 '22 04:09 proceane

@proceane CI is failed

zjffdu avatar Sep 06 '22 08:09 zjffdu

@proceane You'd better change the status of this PR from open to draft.

Oh, I changed it :-)

jongyoul avatar Sep 06 '22 09:09 jongyoul

@proceane You will find detailed information on why CI fails in logs. E.g.

Dependency convergence error for commons-net:commons-net:3.8.0 paths to dependency are:
+-org.apache.zeppelin:zeppelin-server:0.11.0-SNAPSHOT
  +-org.seleniumhq.selenium:selenium-java:2.48.2
    +-org.seleniumhq.selenium:selenium-htmlunit-driver:2.48.2
      +-net.sourceforge.htmlunit:htmlunit:2.60.0
        +-commons-net:commons-net:3.8.0
and
+-org.apache.zeppelin:zeppelin-server:0.11.0-SNAPSHOT
  +-org.apache.hadoop:hadoop-common:2.7.7
    +-commons-net:commons-net:3.1

jongyoul avatar Sep 06 '22 10:09 jongyoul

Long time no see. I think you'd better rebase this branch, not mering current master :-)

jongyoul avatar Feb 16 '23 13:02 jongyoul