Skip to content

GH-3505: Bump sis from 1.4 to 1.5#4015

Merged
afs merged 2 commits into
apache:mainfrom
ThomasThelen:issue_3505
Jun 24, 2026
Merged

GH-3505: Bump sis from 1.4 to 1.5#4015
afs merged 2 commits into
apache:mainfrom
ThomasThelen:issue_3505

Conversation

@ThomasThelen

@ThomasThelen ThomasThelen commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

GitHub issue resolved #3505

Pull request Description:

This PR bumps the SIS version from 1.4 to 1.5. There are several small changes that came along with it.

  1. New derby dependency: SIS 1.5 introduced derby as a dependency, which is why we now add it to the geosparql pom file.
  2. Precision changes: Like most other SIS releases, the floating point values in several computations have changed; these should be visible in the diff
  3. SIS has switched over their CRS conventions from Datum to an Ensemble, CRS:84 being one of them. Rather than constructing the WKT string, we can now call CRS.forCode("CRS:84");

As for new documentation... I don't think any of it needs to change unless the new derby dependency changes anything.


  • Tests are included.
  • Documentation change and updates are provided for the Apache Jena website
  • 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.


See the Apache Jena "Contributing" guide.

@afs afs changed the title GH-3505 Bump sis from 1.4 to 1.5 GH-3505: Bump sis from 1.4 to 1.5 Jun 23, 2026
@afs

afs commented Jun 23, 2026

Copy link
Copy Markdown
Member

Hi @ThomasThelen,

Thank you for this PR and the changes of note.

SIS 1.6 was recently released and it contains a security fix. If there isn't a reason not to, would be possible to update this PR to 1.6? I changed the POM declaration and it at least successfully built and run the tests with no changes other than version.

@ThomasThelen

Copy link
Copy Markdown
Contributor Author

Good call. I bumped the version and the new tests passed without issue. Thanks for taking a look!

@afs afs merged commit 6e14cdf into apache:main Jun 24, 2026
@afs

afs commented Jun 24, 2026

Copy link
Copy Markdown
Member

Thank you!

@afs

afs commented Jun 24, 2026

Copy link
Copy Markdown
Member

Just FYI: Nothing to worry about.

Merging this triggered a security alert on Derby. Its's automated.

While Derby is only used for the development part of the project, not part of the release used in production, we try to stay current so security scans by downstream users are as clean as possible.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update to SIS 1.5

2 participants