[TINKERPOP-1012] BulkLoaderVertexProgram shouldn't assume vertex IDs of type Long

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.1.0-incubating
• Fix Version/s: 3.1.1-incubating
• Component/s: process
• Assignee: Daniel Kuppitz
• Reporter: Dan LaRocque
• Created: 01/Dec/15 12:21
• Updated: 01/Dec/15 18:32
• Resolved: 01/Dec/15 18:32

Description

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.