I think groovy-sql was there because there was a time when I was into polyglot data transforms in Gremlin and groovy-sql enabled you to connect to JDBC data sources in a nice way. So, I included groovy-sql as a convenience. I'm not against removing it, though I think we should remove it in 3.4.0 only in case someone is currently depending on it indirectly. Please try to double check that we aren't using groovy-sql in any of the documentation for some odd example or something.
On Thu, Sep 13, 2018 at 5:20 PM Robert Dale <[email protected]> wrote: > gremlin-driver has a direct dependency on groovy-sql. I removed it in > master and all tests pass (docker/build.sh -i -t -n) so there does not > appear to be any direct or indirect usage. > > Why is it there? Looks like it was pulled in because of > https://issues.apache.org/jira/browse/TINKERPOP-713 > However, groovy-sql does not appear to be a dependency of groovy-console or > anything groovy (except groovy-all). Wasn't then, isn't now. > > Any objections to removing it? > > -- > Robert Dale >
