[
https://issues.apache.org/jira/browse/TINKERPOP3-1012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15033962#comment-15033962
]
ASF GitHub Bot commented on TINKERPOP3-1012:
--------------------------------------------
Github user okram commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/163#issuecomment-161016243
Easy enough. VOTE +1.
@dkuppitz -- when we get the third VOTE, can you be responsible for merging
to master/ and CHANGELOG stuffs?
> BulkLoaderVertexProgram shouldn't assume vertex IDs of type Long
> ----------------------------------------------------------------
>
> Key: TINKERPOP3-1012
> URL: https://issues.apache.org/jira/browse/TINKERPOP3-1012
> Project: TinkerPop 3
> Issue Type: Bug
> Components: process
> Affects Versions: 3.1.0-incubating
> Reporter: Dan LaRocque
> Assignee: Daniel Kuppitz
> Fix For: 3.1.1-incubating
>
>
> BulkLoaderVertexProgram has two lines where it declares a vertex ID variable
> of type Long. While testing bulk loading into a TP impl that uses non-Long
> object IDs with {{userSuppliedIds(false)}}, I found that I needed to change
> the variable type declaration on both of these lines to Object to avoid cast
> exceptions that otherwise killed BLVP. However, I am *not* certain that
> these changes are correct. It's also possible that I just misconfigured
> BLVP. So, I'll submit a PR with the declarations that I changed, but it
> would definitely need to be reviewed by somebody familiar with BLVP.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
