search

RedisGraph / redisgraph-py

RedisGraph python client
https://redisgraph.io
BSD 3-Clause "New" or "Revised" License
189 stars 49 forks source link

Adding merge functionality #19

Closed K-Jo closed 3 years ago

CLAassistant commented 5 years ago

CLA assistant check
All committers have signed the CLA.

Glitchfix commented 5 years ago

Hi @K-Jo , I am not pretty sure if this could work as relations are not the same as edges in this case. Creating a node id to node id relation i.e. creating an edge doesn't work with MERGE query, which happens to be a weird case. This is why I created a separate method to handle it in https://github.com/RedisLabs/redisgraph-py/pull/17

boris-42 commented 4 years ago

IMHO this won't solve the problem, we should adjust code to use MERGE + SET for node updates, I created issue here: https://github.com/RedisGraph/redisgraph-py/issues/81