Unexpected results when updating property about goblin HOT 5 OPEN

I expect this is due to the lack of meta/multi-card props in Neptune and/or the version of Goblin you are using. I am pretty sure this behaves as expected with TinkerGraph and JanusGraph with the newest Goblin.

I haven't had much time to work on these libraries lately and I don't have access to Neptune. I would love to get Goblin integration going for this provider but I may need some community support here...

from goblin.

Sure. I will see if I can make any progress on it, and report back if I do. Was just confirming the desired behavior here before I started to mess with things. The goal of that function is to drop() all properties off of the vertex, and then add them back in? Instead of doing an inline update?

from goblin.

This does not seem Neptune-specific as I've started getting this incorrect behavior after a DSE upgrade as well. I'm guessing it involves a bug in TinkerPop though, because the gremlin bytecode that is created by goblin/gremlinpython looks good, e.g. when updating properties of an existing vertex:

[
 ['addV', 'some_vertex_label'],
 ['property', binding[k0=canonical_name], binding[v0=new name]],
 ['property', binding[k1=names], binding[v1=third name]],
 ['property', <Cardinality.list_: 1>, binding[k2=names], binding[v2=fourth name]], 
 ['property', <Cardinality.list_: 1>, binding[k3=names], binding[v3=fifth name]]
]

This should let the addProperty step behave as described in the TP docs:
http://tinkerpop.apache.org/docs/current/reference/#vertex-properties

The result however is "concat" behavior, rather than overwrite:

{'id': '0c5b6974-ac51-4b78-955c-859eec3b3544', 
 'canonical_name': 'new name', 
 'names': ['first name', 'second name', 'third name', 'fourth name', 'fifth name']
}

The DSE update I did leads me to believe that the addProperty behavior changed somewhere between TP 3.2.x and 3.2.7, but I am still investigating this.

@mbbfyb which version of TinkerPop is used by the Neptune version you are using for these tests?

from goblin.

I will look into this with more detail when I get a spare moment.

from goblin.

@aolieman According to the FAQs it looks like it supports Tinkerpop 3.3

Amazon Neptune’s Gremlin server will support clients that are compatible with Apache TinkerPop version 3.3 using both Websocket and REST connections.

from goblin.