JanusGraph / janusgraph

JanusGraph: an open-source, distributed graph database
https://janusgraph.org
Other
5.25k stars 1.16k forks source link

Change sleepycat.je dependency to a something available in Maven Central #1621

Open patduin opened 5 years ago

patduin commented 5 years ago

Hi, minor issue and there is some background in this issue: https://github.com/odpi/egeria/issues/1116

It would be helpful (if possible) to bump the sleepycat.je dependency to something that's available in Maven Central so downstream projects don't have to add the Oracle repository.

planetf1 commented 5 years ago

As @patduin mentioned we came across this in the build of ODPi Egeria

In our case the dependency tree from mvn dependency:tree is

[INFO] org.odpi.egeria:graph-repository-connector:jar:1.1-SNAPSHOT
[INFO] +- org.janusgraph:janusgraph-berkeleyje:jar:0.3.1:compile
[INFO] |  \- com.sleepycat:je:jar:7.4.5:compile

We are using berkeley as the backing store in a simple test environment (ootb) for our graph. We only express a dependency on 0.3.1 & don't explicitly refer to sleepycat version

The additional repo is being pulled in my the repository definition in the org.janusgraph:janusgraph-berkeleyje pom ie from https://search.maven.org/artifact/org.janusgraph/janusgraph-berkeleyje/0.3.1/jar

<repositories>
        <repository>
            <id>oracleReleases</id>
            <name>Oracle Released Java Packages</name>
            <url>http://download.oracle.com/maven</url>
            <layout>default</layout>
        </repository>
    </repositories>

In @patduins case the enterprise had the maven central repo, but no access to the Oracle repo.

With sleepycat now on maven central ie the latest at https://search.maven.org/artifact/com.sleepycat/je/18.3.12/jar perhaps now would be a good time to consider switching and benefit from dropping this additional repo