[jira] [Commented] (TINKERPOP-2460) Change groovy to provided scope in gremlin-driver

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TINKERPOP-2460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17225469#comment-17225469 ] 

ASF GitHub Bot commented on TINKERPOP-2460:
-------------------------------------------

spmallette merged pull request #1352:
URL: https://github.com/apache/tinkerpop/pull/1352


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Change groovy to provided scope in gremlin-driver
> -------------------------------------------------
>
>                 Key: TINKERPOP-2460
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2460
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: driver
>    Affects Versions: 3.5.0
>            Reporter: Stephen Mallette
>            Assignee: Stephen Mallette
>            Priority: Minor
>              Labels: breaking
>             Fix For: 3.5.0
>
>
> The only reason we have a groovy in {{gremlin-driver}} is to support {{JsonBuilder}} serialization which (1) I tend to doubt is even used and (2) isn't a defined type for any other serialization format besides GraphSON. While it would be nice to simply remove it completely, I think it would be better to give it {{provided}} scope and force users who depend on it currently to directly depend on groovy for it. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)