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

Payara 6: Move all dependencies to stable jakartified versions #9778

Closed
6 of 9 tasks
poikilotherm opened this issue Aug 14, 2023 · 6 comments
Closed
6 of 9 tasks

Payara 6: Move all dependencies to stable jakartified versions #9778

poikilotherm opened this issue Aug 14, 2023 · 6 comments
Labels
D: Payara 6 Upgrade Issues and PRs about the move to Jakarta EE 10 + Payara 6

Comments

@poikilotherm
Copy link
Contributor

poikilotherm commented Aug 14, 2023

@poikilotherm poikilotherm added the D: Payara 6 Upgrade Issues and PRs about the move to Jakarta EE 10 + Payara 6 label Aug 14, 2023
@poikilotherm
Copy link
Contributor Author

Looking at the dependency tree, there are two potentially broken javax dependencies:

  1. Duracloud reportdata uses XML binding (javax.xml.bind)
  2. Google Cloud Storage uses javax.annotations

@qqmyers are these code paths used by us? Can we test for those?

@qqmyers
Copy link
Member

qqmyers commented Aug 14, 2023

Those are both in archivers. Duracloud is used by Texas and could probably be tested with them. The Google archiver is used by QDR and could be tested with them or by anyone with a Google account. FWIW, QDR already uses different google dependencies than the main branch - it might be a good quick test to upgrade to those and see if the javax issues are still in what's pulled in.

@poikilotherm poikilotherm added this to the 6.0 milestone Aug 14, 2023
@cmbz
Copy link

cmbz commented Aug 16, 2023

Action:

  • Split this issue into several issues and size each
  • Team uses issues as learning opportunity, consulting with @qqmyers and @poikilotherm as needed to make progress and ensure that other stakeholders are not unduly affected by changes
  • Contact @poikilotherm to confirm sizing for each issue

@pdurbin
Copy link
Member

pdurbin commented Aug 16, 2023

Specifically we talked about taking these two items...

  • SWORD2 lib needs a release and no snapshot
  • XOAI lib needs a release and no snapshot

... and creating an issue to document in the dev guide how to release these to Maven Central. It would be a learning opportunity for someone (or someones) on the team. We'll need the password, of course.

@pdurbin pdurbin self-assigned this Aug 17, 2023
@pdurbin pdurbin closed this as completed Aug 21, 2023
@pdurbin pdurbin removed this from the 6.0 milestone Aug 21, 2023
@pdurbin pdurbin removed their assignment Aug 21, 2023
@scolapasta scolapasta modified the milestone: 6.0 Aug 21, 2023
@poikilotherm
Copy link
Contributor Author

FWIW: I looked into the potential SFL4J thing. It doesn't seem to be an issue anymore! So I never created an issue for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D: Payara 6 Upgrade Issues and PRs about the move to Jakarta EE 10 + Payara 6
Projects
None yet
Development

No branches or pull requests

5 participants