zeppelin
zeppelin copied to clipboard
[ZEPPELIN-6084] Update jena-arq to 4.10
What is this PR for?
This PR is a follow-up to #4507. I have decided to switch to the latest version 4.x. From 5.x JDK 17 is required. Obsolete functions have been removed.
Note: I do not use the interpreter myself. I have not tested it manually either. I rely on the unit tests.
What type of PR is it?
Improvement
What is the Jira issue?
- https://issues.apache.org/jira/browse/ZEPPELIN-6084
How should this be tested?
- CI
Questions:
- Does the license files need to update? No
- Is there breaking changes for older versions? Maybe
- Does this needs documentation? No
@pjfanning Can you do a review here?
@645775992 Can you review this PR?
Jena is not something I know a great deal about but the changes look ok to me.
@pan3793 @jongyoul Please review.
It seems that the sparql interpreter does not have integration tests, I don't use sparql, so may not be able to evaluate this change. Is it possible to leverage the testcontainers to build an integration test?