Github user dkuppitz commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/107#issuecomment-148097019
Wow, multi-valued properties really suck. I added a test case for the Crew
graph, only to realize that multi-valued properties aren't handled properly by
BLVP. I thought it's easy to fix, but it's not.
The default implementation of `IncrementalBulkLoader` doesn't know anything
about a schema. Now, if it's about to write a property, how should it know,
whether it should overwrite an existing value or simply append the new value? I
don't think there's a general solution and multi-valued properties need to be
handled domain specific.
`IncrementalBulkLoader` should always assume single-valued properties.
Thoughts?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
