Skip to content

Payara7.2026.6 Plus Multiple Library updates#12498

Draft
qqmyers wants to merge 8 commits into
IQSS:developfrom
QualitativeDataRepository:payara7.2026.6plus
Draft

Payara7.2026.6 Plus Multiple Library updates#12498
qqmyers wants to merge 8 commits into
IQSS:developfrom
QualitativeDataRepository:payara7.2026.6plus

Conversation

@qqmyers

@qqmyers qqmyers commented Jun 30, 2026

Copy link
Copy Markdown
Member

What this PR does / why we need it: This PR updates multiple libraries used in Dataverse along with updating to Payara 7.2026.6. I'm planning to use all of these at QDR, but whether these should all be updated for 6.12 or not is tbd.

Which issue(s) this PR closes:

  • Closes #

Special notes for your reviewer: There are a couple fixes for tests to pass because the updated libraries pull in a new XML processor. Using TransformerFactory.newDefaultInstance(); keeps the current one. Alternately we could use the new processor (TransformerFactory.newInstance()) and address the changes (it seems to be hardcoded at 3 spaces indent for pretty printing and I haven't checked for other differences.)
The PR also includes a couple test changes to allow them to pass on Windows - they were hardcoded to a specific line separator or the existence of a /tmp dir.

Lastly, there's an odd comment about jhove 1.34 in the PR - when I initially tried updating to it, there was a commons-compress conflict as some of the jhove modules pulled in an old version. I eventually solved that (by pulling in commons-compress first), but eventually decided to revert to the current jhove. (jhove 1.34 doesn't yet have all the modules packaged for maven and the 1.30 version separated the modules into separate maven dependencies. 1.20.1 is old, but for QDR I decided to stop working on it for now.)

Suggestions on how to test this: Regression testing - nothing should be changed with these updates.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?: release notes would need the normal payara version update instructions.

Additional documentation:

@coveralls

coveralls commented Jun 30, 2026

Copy link
Copy Markdown

Coverage Status

Coverage is 25.024%QualitativeDataRepository:payara7.2026.6plus into IQSS:develop. No base build found for IQSS:develop.

@pdurbin pdurbin added this to the 6.12 milestone Jul 8, 2026
@Ozoubia

Ozoubia commented Jul 10, 2026

Copy link
Copy Markdown

Hi @qqmyers,
following the topic on Zulip (#containers > Upgrading payara only), I have a question regarding the upgrade. We built the base image from the plain v6.11 tag with only -Dpayara.version=7.2026.6, without any of the code/library changes from this PR. It deploys cleanly on our test instance and basic smoke tests (login, upload, search, metadata exports) pass.
The Payara version was flagged by our IT security, so we cannot wait for the next Dataverse release.
Do you know of anyone running Payara 7.2026.6 in production? And are any of the changes here required at runtime? We're planning to run this combination in production until 6.12 ships.
Thank you in advance :)

@pdurbin

pdurbin commented Jul 10, 2026

Copy link
Copy Markdown
Member

-Dpayara.version=7.2026.6

For a bit more context, this comes from https://guides.dataverse.org/en/6.11/container/base-image.html#build-instructions

Screenshot 2026-07-10 at 6 40 55 AM

@qqmyers

qqmyers commented Jul 10, 2026

Copy link
Copy Markdown
Member Author

The code changes here were due to other library updates, so I suspect that you'd be fine just deploying the newer Payara. You could update the payara bom version and rebuild the Dataverse war, but I'd usually expect compatibility across minor version updates. QDR often uses Payara versions newer than the recommended and have only encountered issues (which then get fixed in PRs we submit) when Payara or one of the libraries they use has had a major change.

@pdurbin

pdurbin commented Jul 10, 2026

Copy link
Copy Markdown
Member

I just checked with @qqmyers and now I understand better that this PR could have only included an update for Payara. He just happened to include some unrelated library updates plans to roll out.

@Ozoubia

Ozoubia commented Jul 13, 2026

Copy link
Copy Markdown

that's good to know, thank you for the help :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants