Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review Jena dependencies in CWLViewer #498

Open
kinow opened this issue Jan 5, 2023 · 1 comment
Open

Review Jena dependencies in CWLViewer #498

kinow opened this issue Jan 5, 2023 · 1 comment

Comments

@kinow
Copy link
Member

kinow commented Jan 5, 2023

Description

It appears we are using Jena 3.x API, even though we have Jena 4.x in the dependencies. It's being brought into the code via Taverna: apache/jena#1293 (comment)

Expected Behavior

Use latest Jena 4.x

Actual Behavior

Possible Fix

Use updated API.

Steps to Reproduce

Context

Your Environment

  • Version used:
  • Environment name and version (e.g. Chrome 39, Java 17):
@kinow
Copy link
Member Author

kinow commented Jan 6, 2023

See comments in the linked issue in jena, e.g.

Not sure if CWLViewer is using Jena 4 and 3 at the same time.

Jena uses ServiceLoader to initialize so if jars from both on the classpath, then, yes, it is both (jena-core from v4, RIOT from v3). Since 3.0.1, RIOT plugged itself into jena-core as an RDFReader.

This is (not) going to be fun to tackle this issue, I think.

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

No branches or pull requests

1 participant