Skip to content

Conversation

@Reamer
Copy link
Contributor

@Reamer Reamer commented Dec 18, 2024

What is this PR for?

This PR updates Jetty and some subcomponents such as Jackson, Jersey, Micrometer, Kubernetes client.

In my opinion, it is not possible to split the pull request due to the annotation change.

What type of PR is it?

  • Improvement

What is the Jira issue?

How should this be tested?

  • CI
  • - Manually

Questions:

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

@Reamer Reamer changed the title Jakarta [ZEPPELIN-6158] Update Jetty and other libraries to use Jakarta Dec 18, 2024
@Reamer
Copy link
Contributor Author

Reamer commented Dec 18, 2024

Ready-for-Review

@pjfanning
Copy link
Member

@jongyoul would you be able to review this?

@Reamer
Copy link
Contributor Author

Reamer commented Mar 26, 2025

We are currently running into the following Selenium bug. I think we can therefore ignore the failed tests.
SeleniumHQ/selenium#15358

@Reamer Reamer requested review from jongyoul and pjfanning and removed request for pjfanning March 26, 2025 08:11
Copy link
Member

@pjfanning pjfanning left a comment

Choose a reason for hiding this comment

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

lgtm

@jongyoul
Copy link
Member

@Reamer I'm sorry for the late reply, and thank you for the hard work. I left two comments. Could you please check it?

@Reamer Reamer force-pushed the jakarta branch 2 times, most recently from c70c53a to 686a135 Compare April 11, 2025 08:44
@Reamer Reamer merged commit e928edb into apache:master Apr 17, 2025
16 of 17 checks passed
asfgit pushed a commit that referenced this pull request Apr 17, 2025
### What is this PR for?
This PR updates Jetty and some subcomponents such as Jackson, Jersey, Micrometer, Kubernetes client.

In my opinion, it is not possible to split the pull request due to the annotation change.

### What type of PR is it?
 - Improvement

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-6158

### How should this be tested?
* CI
* [x] - Manually

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

Closes #4902 from Reamer/jakarta.

Signed-off-by: Philipp Dallig <[email protected]>
(cherry picked from commit e928edb)
Signed-off-by: Philipp Dallig <[email protected]>
@Reamer Reamer deleted the jakarta branch April 17, 2025 09:11
@Reamer
Copy link
Contributor Author

Reamer commented Apr 17, 2025

Merged into master and branch-0.12

basapuram-kumar pushed a commit to acceldata-io/zeppelin that referenced this pull request Jun 25, 2025
This PR updates Jetty and some subcomponents such as Jackson, Jersey, Micrometer, Kubernetes client.

In my opinion, it is not possible to split the pull request due to the annotation change.

 - Improvement

* https://issues.apache.org/jira/browse/ZEPPELIN-6158

* CI
* [x] - Manually

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

Closes apache#4902 from Reamer/jakarta.

Signed-off-by: Philipp Dallig <[email protected]>
prabhjyotsingh pushed a commit to acceldata-io/zeppelin that referenced this pull request Jul 3, 2025
…arta

This PR updates Jetty and some subcomponents such as Jackson, Jersey, Micrometer, Kubernetes client.

In my opinion, it is not possible to split the pull request due to the annotation change.

 - Improvement

* https://issues.apache.org/jira/browse/ZEPPELIN-6158

* CI
* [x] - Manually

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

Closes apache#4902 from Reamer/jakarta.

Signed-off-by: Philipp Dallig <[email protected]>
(cherry picked from commit e928edb)
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.

3 participants