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

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

Github user spmallette commented on a diff in the pull request:

    https://github.com/apache/incubator-tinkerpop/pull/181#discussion_r48017013
  
    --- Diff: docs/src/reference/the-graphcomputer.asciidoc ---
    @@ -364,10 +364,11 @@ It's recommended to tune this property for the target 
graph and not use the defa
     |`writeGraph(String)` | Sets the path to a `GraphFactory` compatible 
configuration file for the target graph. | _none_
     |========================================
     
    -NOTE: `BulkLoaderVertexProgram` comes with a default `BulkLoader` 
implementation, namely `IncrementalBulkLoader`. It
    -will work for the most use-cases, but has one limitation though: It 
doesn't support multi-valued properties.
    -`IncrementalBulkLoader` will handle every property as a single-valued 
property. A custom `BulkLoader` implementation
    -has to be used if the default behavior is not sufficient.
    +NOTE: `BulkLoaderVertexProgram` uses the `IncrementalBulkLoader` by 
default. The other option is the `OneTimeBulkLoader`,
    +which doesn't store any temporary IDs in the `writeGraph` and thus should 
only be used for initial bulk loads. Both
    +implementations should cover the mahority of use-cases, but have a 
limitation though: They don't support multi-valued
    --- End diff --
    
    spelling mistake - should be "majority" :smile: 


> BVLP should store vertex IDs as String
> --------------------------------------
>
>                 Key: TINKERPOP-1026
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1026
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.0.2-incubating
>            Reporter: Daniel Kuppitz
>            Assignee: Daniel Kuppitz
>             Fix For: 3.1.1-incubating
>
>
> Vertex IDs can be types that are not necessarily supported as a property 
> type, hence BLVP should store the {{toString()}} variant of vertex IDs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to