Skip to content

Pva/upgrade java 21#456

Open
peter-va wants to merge 96 commits intodevelopfrom
PVA/Upgrade-Java-21
Open

Pva/upgrade java 21#456
peter-va wants to merge 96 commits intodevelopfrom
PVA/Upgrade-Java-21

Conversation

@peter-va
Copy link
Copy Markdown
Collaborator

@peter-va peter-va commented Feb 6, 2026 •
edited
Loading

Upgrade to Java 21:

  1. Remove unused travis.yml and Dockerfile.
  2. Update workflow yamls to use Java 21.
  3. Update build.gradle to use Java 21.
  4. Migrate deprecated/renamed javax libraries to jakarta and update to 4.0.x
  5. Change deprecated Schema required to Schema.requiredMode.
  6. Force UTF-8 file reading for consistencies sake since >Java 18 assumes it.
  7. Migrate deprecated Java.net.URL to Java.net.URI.create().toURL(), both main and test.
  8. Added small test for getValueFromFile since it's a backbone function that gets changed.

Testing:

  • Run unit tests before migration (handful of tests broken)
  • Run unit tests after migration (the same ones as before are broken, no extras)
  • Run devreset through without errors
  • Run unit tests again (only 2 FHIR tests left broken)
  • Manual testing

bcarlsenca and others added 30 commits October 17, 2025 10:22
@bcarlsenca
Set version to 2.4.0
891ad6a
@bcarlsenca
Merge branch 'develop' into develop-2.4
7469ac0
@bcarlsenca
EVSRESTAPI-681: avoid re-running for the same terminology (#428)
4e78c7b
@shapirod2
Dss/evsrestapi 634 search modifiers (#429)
7d55530 
* EVSRESTAPI-636: spotless misc

* EVSRESTAPI-634: uri search modifiers and string missing search modifiers
fix string -> integer
488d844
EVSRESTAPI-678
da796e6
spotless
1952348
@shapirod2
Dss/evsrestapi 634 search modifiers (#431)
87ff970 
* EVSRESTAPI-636: spotless misc

* EVSRESTAPI-634: uri search modifiers and string missing search modifiers

* EVSRESTAPI-634: same search modifiers for R4

* EVSRESTAPI-634: spotless misc
most changes, need to filter remodeled properties
550be09
@peter-va
Merge pull request #430 from NCIEVS/PVA/EVSRESTAPI-495-fix-fromRecord…
98d9414 
…-pageSize

Pva/evsrestapi 495 fix from record page size
ignore remodeled properties
8f04130
spotless fix
5ca0b1e
more thorough test
6c05b01
@shapirod2
EVSRESTAPI-679: nci term forms attachment and tests for ncit (refacto…
29ad429 
…r for cdisc) (#434)
@shapirod2
Dss/evsrestapi 671 prefix docs (#435)
2aa1af0 
* EVSRESTAPI-671: sparql prefix documentation

* EVSRESTAPI-671: spotless misc.
@peter-va
Merge pull request #433 from NCIEVS/PVA/EVSRESTAPI-220-property-value…
c423252 
…s-endpoint

Pva/evsrestapi 220 property values endpoint
@akuppusamy-wci
[EVSRESTAPI-677] Fix issue with TermSuggestionFormControllerTests (#432)
82a9066 
* [EVSRESTAPI-574] Improve handling of the triplestore databases - NCIT2, CTRP

* [EVSRESTAPI-677] Fix issue with TermSuggestionFormControllerTests

* [EVSRESTAPI-677] Fix issue with TermSuggestionFormControllerTests
@bcarlsenca
Merge branch 'develop' into develop-2.4
c949a1d
@shapirod2
EVSRESTAPI-679: nci term forms attach (#436)
e9d73c9
@bcarlsenca
Merge branch 'develop' into develop-2.4
325cdce
@shapirod2
Dss/evsrestapi 689 todos (#443)
effe281 
* EVSRESTAPI-689: GraphReportLoadServiceImpl.java log hierarchy info

* EVSRESTAPI-689: removing TODOs; some minor test adjustments

* EVSRESTAPI-689: resolve more todos

* EVSRESTAPI-689: more notes

* spotless misc
@shapirod2
Dss/evsrestapi 690 property param
d100165 
adding property parameter for CodeSystem$lookup operation
migrate ObjectMapper to ThreadLocalMapper
d3ed6d8
fix what tests i can
db805fe
@shapirod2 @bcarlsenca
EVSRESTAPI-671: sparql prefix related documentation (#439)
0525ddf 
* EVSRESTAPI-671: sparql prefix related documentation

* Fix formatting

---------

Co-authored-by: Brian Carlsen <bcarlsen@westcoastinformatics.com>
@bcarlsenca
Merge branch 'develop' into develop-2.4
31bb775
Merge branch 'develop-2.4' into PVA/EVSRESTAPI-684-ThreadLocalMapper-…
b6efe6c 
…replace
spotbugs
2f2373f
various bugfixes from spotbugs
80149b3
fix filepath
02de0ea
@peter-va peter-va marked this pull request as ready for review February 12, 2026 01:12
@peter-va peter-va requested a review from bcarlsenca February 12, 2026 01:12
Base automatically changed from develop-2.4 to develop March 3, 2026 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bcarlsenca bcarlsenca Awaiting requested review from bcarlsenca

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@peter-va @bcarlsenca @shapirod2 @akuppusamy-wci