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

AMBARI-26309: Ambari server takes up about an hour to start #3950

Closed
wants to merge 1 commit into from

Conversation

guluo2016
Copy link
Contributor

@guluo2016 guluo2016 commented Mar 8, 2025

What changes were proposed in this pull request?

Issue details see: AMBARI-26309

There are several changes in this PR, as following.

  1. Upgrade eclipselink to 4.0.5.
    This change mainly solves the problem of long-time hanging when starting ambari-server.
    This issue does exists in eclipselink 2.7.14, issue details see: Slow startup because of low entropy for PRNG eclipse-ee4j/eclipselink#2075
    And this issue was fixed in eclipselink 4.0.5, details see: [4.0] Fix #2075 Slow startup because of low entropy for PRNG (#2318) eclipse-ee4j/eclipselink#2335

  2. Upgrade to JPA 3.x
    eclipselink 4.x follows JPA 3.x specification.
    And an important change in JPA 3 is the change in package names, from javax.persistence to jakarta.persistence.
    So, I have added the jakarta.persistence-api-3.1.0 to ambari project, and update package name from javax.persistence to jakarta.persistence.

  3. Upgrade guice to 7.0.0
    The reason: guice didnot add support for jakarta.inject until version 7.0.0, issue details see: jakarta.inject support google/guice#1383, and Guice 7.0.0 changes see: https://github.com/google/guice/wiki/Guice700#:~:text=JEE%20Jakarta%20Transition,7.0%20are%20identical

How was this patch tested?

I have already manually tested it, amabri-server can start and run normally.

Please review Ambari Contributing Guide before opening a pull request.

@JiaLiangC
Copy link
Contributor

@guluo2016 The issues encountered in your PR have already been addressed and fixed in the Spring 6 upgrade branch. This is a duplicate submission.

@guluo2016
Copy link
Contributor Author

Thank you for your reminder. I have seen the relevant PR , so this PR will be closed.
Thanks!

@guluo2016 guluo2016 closed this Mar 23, 2025
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