Skip to content

Conversation

@afs
Copy link
Member

@afs afs commented Oct 20, 2025

GitHub issue resolved #3528

Includes some final clearing up from jena-iri and switching to having only jena-iri3986.


  • Tests are included.
  • Commits have been squashed to remove intermediate development commit messages.
  • Key commit messages start with the issue number (GH-xxxx)

By submitting this pull request, I acknowledge that I am making a contribution to the Apache Software Foundation under the terms and conditions of the Contributor's Agreement.

@afs afs added the Jena6 Changes relating to Jena6 label Oct 20, 2025
@afs
Copy link
Member Author

afs commented Oct 21, 2025

@Aklakan -- I hope that there are no changes that impact any of your current PRs.

The changes are POMs, deal with warnings from jena 21, especially new URL() used in tests, some remaining uses of jena-iri were lurking. Object.finalize is now deprected for removal. In geosparql, the override of TaskThread.isTerminated no longer is legal.

Subject to more testing, I think the switch-over is ready.

@Aklakan
Copy link
Contributor

Aklakan commented Oct 22, 2025

Thanks for the heads up @afs - is there anything from my side blocking your PR? If not I would just rebase after your merge into main and fix any conflicts.

@afs
Copy link
Member Author

afs commented Oct 22, 2025

is there anything from my side blocking your PR? If not I would just rebase after your merge into main and fix any conflicts.

No - nothing blocking.

I've been merging it into various branches I have with no problems for Java.

Changes to POMs from simplifying the test dependencies in parallel to changing the version caused merge conflicits but nothing complicated.

The less change at the switchover point the better, so that further work (e.g. deprecation removals) can be done in isolated steps.

@afs afs merged commit a919b5e into apache:main Oct 23, 2025
@afs afs deleted the setup-jena6 branch October 23, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Jena6 Changes relating to Jena6

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Jena6 development

2 participants