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

Initial support for the 'jena' candidate #615

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

berezovskyi
Copy link

Jena and RDF4J are the two most popular semantic web libraries. The CLI tools include converting RDF formats, querying graphs ad-hoc and loading large files into the DB. brew install jena will give you an install to explore:

bin $ ls
arq	      juuid	rdfcompare  riot       shacl	       tdb2.tdbcompact	tdb2.tdbupdate	tdbloader2  turtle  wwwenc
infer	      nquads	rdfcopy     rset       shex	       tdb2.tdbdump	tdb2.xloader	tdbquery    uparse  xload-common
iri	      ntriples	rdfdiff     rsparql    sparql	       tdb2.tdbloader	tdbbackup	tdbstats    update  xload-data
jena	      qparse	rdfparse    rupdate    tdb1.xloader    tdb2.tdbquery	tdbdump		tdbupdate   utf8    xload-index
jena.version  rdfcat	rdfxml	    schemagen  tdb2.tdbbackup  tdb2.tdbstats	tdbloader	trig	    wwwdec  xload-main

Used ActiveMQ migration as a template. This is my first PR to SDKMAN, looking forward to your feedback.

Signed-off-by: Andrew Berezovskyi <andriib@kth.se>
@berezovskyi berezovskyi changed the title Initial support for 'jena' candidate Initial support for the 'jena' candidate Nov 17, 2022
Copy link
Member

@marc0der marc0der left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just wanted to clarify if you were also a maintainer of this project. The alternative way of adding versions to the DB is through our Vendor API. But this requires that you are an active maintainer to Apache Jena.

def migration001(implicit db: MongoDatabase): Candidate = {
Candidate(
candidate = CandidateName,
name = "Apache Jena",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's just drop the Apache here for brevity.

import com.mongodb.client.MongoDatabase
import org.bson.Document

@ChangeLog(order = "100")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When last I counted, we only had 73.

candidate = CandidateName,
name = "Apache Jena",
description =
"Apache Jena (or Jena in short) is a free and open source Java framework for building semantic web and Linked Data applications. The framework is composed of different APIs and CLI tools interacting together to process RDF data.",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd leave out the bit in brackets (or Jena in short).

@berezovskyi
Copy link
Author

Thank you, @marc0der. I just updated the PR as you requested.

I am not a Jena maintainer but can I reach out to the dev mailing list if you think the Vendor API (https://sdkman.io/vendors I presume) is a better route. Can we consider this a future improvement or do you strongly prefer the use of Vendor API instead?

@berezovskyi
Copy link
Author

@marc0der did you have a chance to check out the changes I did as you requested?

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

Successfully merging this pull request may close these issues.

None yet

2 participants