Excavator: Allow Java 17 language features in all modules

palantir / atlasdb

Transactional Distributed Database Layer

https://palantir.github.io/atlasdb/

Apache License 2.0

55 stars 14 forks source link

Closed svc-excavator-bot closed 2 weeks ago

svc-excavator-bot commented 1 month ago

Changes produced by the roomba/java-17-library-target check.

To enable or disable this check, please contact the maintainers of Excavator.

Also upgrade all references from javax to jakarta.

tillyow commented 1 month ago

Needs a Jakarta migration, blocked

fsamuel-bs commented 2 weeks ago

Documenting for my future self:

upgrading to java 17 fails because we use some dropwizard extensions in tests that use reflection
upgrading to later dropwizard requires javax -> jakarta migration
we rely on some dependencies (from Cassandra 2.x) which still use javax, and not sure we can migrate them
currently last build is failing because we're not finding in the test runtime classpath jakarta.el classes.