zeppelin icon indicating copy to clipboard operation
zeppelin copied to clipboard

[ZEPPELIN-6084] Update jena-arq to 4.10

Open Reamer opened this issue 1 year ago • 5 comments
trafficstars

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

Reamer avatar Oct 28 '24 09:10 Reamer

@pjfanning Can you do a review here?

Reamer avatar Nov 07 '24 07:11 Reamer

@645775992 Can you review this PR?

Reamer avatar Nov 12 '24 12:11 Reamer

Jena is not something I know a great deal about but the changes look ok to me.

pjfanning avatar Nov 12 '24 12:11 pjfanning

@pan3793 @jongyoul Please review.

Reamer avatar Nov 24 '24 12:11 Reamer

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?

pan3793 avatar Nov 25 '24 03:11 pan3793