[ https://issues.apache.org/jira/browse/RYA-405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16443070#comment-16443070 ]
ASF GitHub Bot commented on RYA-405: ------------------------------------ GitHub user ejwhite922 opened a pull request: https://github.com/apache/incubator-rya/pull/291 RYA-405 Migration of OpenRDF Sesame libraries to RDF4J ## Description Continued from [PR-245](https://github.com/apache/incubator-rya/pull/245). This updates Rya from OpenRDF Sesame 2.7.6 to RDF4J 2.3.1. Some notes on what's changed and what to look for when reviewing: - A Sesame to RDF4J migration guide can be found [here](http://docs.rdf4j.org/migration/) - The new libraries changed constant var names from using "-const-" to "\_const\_{a\_unique\_hex\_string}\_{type\_info}" (See org.eclipse.rdf4j.query.algebra.helper.TupleExprs#getConstVarName(Value)). This broke a lot of our code but is now resolved through a new class we added named **VarNameUtils**. All hard-coded "-const-" strings were removed. - We added another new class, **RdfFormatUtils**, to handle the deprecation of RDFFormat#valueOf(String). - NumericLiteralImpl was deprecated. So, we replaced its use with AbstractValueFactory#createLiteral(BigInteger). I'm not sure I like this as it's a little unintuitive for creating an int with type XMLSchema.INTEGER instead of XMLSchema.INT. An alternative could be to use AbstractValueFactory#createLiteral(String, XMLSchema.INTEGER) which isn't much better but makes it clear that it must be XMLSchema.INTEGER. - META-INF/services files had their names updated from org.openrdf.sesame... to org.eclipse.rdf4j... - More accepted classes were added to the deserialize() methods for the following classes (not sure on these but they passed): - org.apache.rya.indexing.pcj.fluo.app.AggregationResultUpdater.ObjectSerializationAggregationStateSerDe - org.apache.rya.indexing.pcj.storage.accumulo.VisibilityBindingSetSerDe - SPIN Construct Rules accept more rules since less bnodes are produced from consequent patterns. This typically leads to more inferences being used but the same results being produced. - Vagrant file updated - Updated org.apache.rya.api.path.PathUtils check for secure directory when adding RyaShellHistoryProvider file (Windows only fix / works for Linux). Not sure if there was a defect for this already. ### Tests Build Unit Tests ### Links [Jira](https://issues.apache.org/jira/browse/RYA-405) ### Checklist - [ ] Code Review - [ ] Squash Commits #### People To Review @isper3at @kchilton2 @pujav65 @DLotts @meiercaleb @amihalik @jessehatfield @jomach You can merge this pull request into a Git repository by running: $ git pull https://github.com/ejwhite922/incubator-rya RYA-405_MigrateSesameToRDF4J_Update Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-rya/pull/291.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #291 ---- commit 434105eab297de4fe7417df8beaa9941639fc667 Author: Jorge Machado <jorge.w.machado@...> Date: 2017-10-18T09:32:39Z RYA-405 Migration of OpenRDF Sesame libraries to RDF4J Co-authored-by: eric.white <eric.wh...@parsons.com> Co-authored-by: Jorge Machado <jorge.w.mach...@hotmail.com> ---- > Migrate from Sesame to rdf4j libs > ---------------------------------- > > Key: RYA-405 > URL: https://issues.apache.org/jira/browse/RYA-405 > Project: Rya > Issue Type: Improvement > Affects Versions: 3.2.12 > Reporter: Jorge Machado > Assignee: Eric White > Priority: Major > Fix For: 4.0.0 > > > I have migrated all the project to the newer libs from rdf4j instead of using > the old ones. > Can someone take a look ? -- This message was sent by Atlassian JIRA (v7.6.3#76005)