search

marklogic / marklogic-jena

Adapter for using MarkLogic with the Jena RDF Framework
Other
5 stars 11 forks source link

upgrade Jena and MarkLogic dependencies #76

Closed ehennum closed 2 years ago

ehennum commented 5 years ago

latest stable production releases

AlexTo commented 5 years ago

hi @ehennum, do you have any build issues after upgrading the dependencies?

ehennum commented 5 years ago

@AlexTo , the MarkLogic Java API dependency was a clean update, but the Jena upgrade had an issue.

The Jena Transactional interface added some methods. marklogic-jena has an implementation of that interface, so the marklogic-jena class implementing Transactional must add some methods.

I need to study up on the Jena transactional model before writing those methods.

AlexTo commented 5 years ago

I thought so too as the last time I attempted to upgrade Jena to 3.7.0 and realised I needed to also update MarkLogic classes accordingly. Eventually I removed support for Jena in our software as I didn't have the time to do so but this update is highly desired :) hopefully MarkLogic team could provide an upgrade soon

ehennum commented 5 years ago

Although the new Transactional methods seem biased toward a specific transactional model, the impacts on marklogic-jena seem straightforward, in that the MarkLogic Java API only supports write transactions for multistatement transactions.

Anyway, the unit tests ran fine after the version upgrade. No new functional tests are needed because the version upgrade doesn't add any new functionality.

If the version upgrade doesn't cause any regressions on the internal Jenkins job that runs the functional tests, I expect it is good as a release candidate.

georgeajit commented 5 years ago

Closing this issue.

AlexTo commented 5 years ago

Hi just checking if there is any plan to release this upgrade yet to support Jena 3.1x.0 ? Thank you

ehennum commented 5 years ago

@AlexTo , the marklogic-jena project is now a community project:

https://github.com/marklogic/marklogic-jena/blob/master/LICENSE#L10

Contrary to our expectations, the project hasn't seen much interest:

https://github.com/marklogic/marklogic-jena/graphs/traffic

AlexTo commented 5 years ago

I pulled the "Dev" branch which has your latest commit and it seems to work ok with Jena 3.12.0 Thanks

fp-smartlogic commented 2 years ago

Hi. Are there any plans of upgrading Jena dependency to 4.x version?

rjrudin commented 2 years ago

This will be addressed in the next release. Thanks to @fp-smartlogic for doing the work of supporting Jena 4.x.