neo4j-contrib / neo4j-tinkerpop-api-impl

Implementation of Apache Licensed Neo4j API for Tinkerpop3
Other
24 stars 15 forks source link

Cyclic dependency on TinkerPop prevents creating release #7

Closed robertdale closed 7 years ago

robertdale commented 7 years ago

@jexp

TinkerPop 3.3.0 is entering code freeze and a soon release.

TinkerPop 3.3.0 (neo4j-gremlin) depends on neo4j-tinkerpop-api-impl (0.7-3.2.2-SNAPSHOT). neo4j-tinkerpop-api-impl (0.7-3.2.2-SNAPSHOT) depends on tinkerpop (3.3.0-SNAPSHOT).

This cyclic dependency makes it complicated to create releases in either project.

It appears that the introduction of a gremlin stored procedure creates the dependency on tinkerpop. Proposed solution is to pull out the gremlin stored proc into its own module to free neo4j-tinkerpop-api-impl from any tinkerpop deps.

If this issue is not resolved, it will be a missed opportunity to upgrade neo4j in tinkerpop until the next major or minor code release.

See also:

robertdale commented 7 years ago

Impl has been isolated and split into separate module.