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

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

Github user PBGraff commented on the issue:

    https://github.com/apache/tinkerpop/pull/772
  
    @spmallette I'm not sure I'll come back to this exactly. I was thinking 
about it recently, actually, and I think just providing a `create()` which 
calls `create(null)` in the VertexProgramBuilder would be useful for Java 
users. This is a simpler change than making it varagrs and less likely to cause 
breaking changes and probably covers 99+% of use cases.


> VertexProgram create with varargs for Graphs
> --------------------------------------------
>
>                 Key: TINKERPOP-1861
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1861
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.2.7
>            Reporter: Philip Graff
>            Priority: Minor
>
> VertexProgram.Builder.create(Graph) can be modified to 
> VertexProgram.Builder.create(Graph...) so that passing in zero or many graphs 
> is naturally handled. The current state of passing in null when no graph is 
> needed is bad practice.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to