You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tinkerpop.apache.org by "dkuppitz (GitHub)" <gi...@apache.org> on 2018/12/05 15:07:48 UTC

[GitHub] [tinkerpop] dkuppitz opened pull request #1011: TINKERPOP-1849 Provide a way to fold() with an index

https://issues.apache.org/jira/projects/TINKERPOP/issues/TINKERPOP-1849

`docker/build.sh -t -i -n` passed and the docs built with `docker/build.sh -d` look good.

VOTE +1

[ Full content available at: https://github.com/apache/tinkerpop/pull/1011 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

[GitHub] [tinkerpop] asfgit closed pull request #1011: TINKERPOP-1849 Provide a way to fold() with an index

Posted by "asfgit (GitHub)" <gi...@apache.org>.

[ pull request closed by asfgit ]

[ Full content available at: https://github.com/apache/tinkerpop/pull/1011 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

[GitHub] [tinkerpop] spmallette commented on pull request #1011: TINKERPOP-1849 Provide a way to fold() with an index

Posted by "spmallette (GitHub)" <gi...@apache.org>.

You mention that impact of doing `index()` on something that is not a collection in the first sentence of this section but I think that it's worth demonstrating as an example so that people connect the relevance of that statement.

[ Full content available at: https://github.com/apache/tinkerpop/pull/1011 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

[GitHub] [tinkerpop] spmallette commented on pull request #1011: TINKERPOP-1849 Provide a way to fold() with an index

Posted by "spmallette (GitHub)" <gi...@apache.org>.

I assume that "There two indexers" should be "There are two indexers"

[ Full content available at: https://github.com/apache/tinkerpop/pull/1011 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

[GitHub] [tinkerpop] spmallette commented on issue #1011: TINKERPOP-1849 Provide a way to fold() with an index

Posted by "spmallette (GitHub)" <gi...@apache.org>.

VOTE +1

[ Full content available at: https://github.com/apache/tinkerpop/pull/1011 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

[GitHub] [tinkerpop] spmallette commented on pull request #1011: TINKERPOP-1849 Provide a way to fold() with an index

Posted by "spmallette (GitHub)" <gi...@apache.org>.

I think you should have something in upgrade docs to introduce this feature so that folks know it is there. 

[ Full content available at: https://github.com/apache/tinkerpop/pull/1011 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org