Skip to content

Conversation

Glitchfix
Copy link
Contributor

@Glitchfix Glitchfix commented Dec 10, 2018

#15
Feature to merge node or relation added

USAGE:

tim = Node(label='person', alias=client.random_string(), properties={'name': 'Tim Doe', 'age': 25, 'gender': 'male', 'status': 'single'})
redis_graph.merge(tim)

tom = Node(label='person', properties={'name': 'Tom Doe', 'age': 23, 'gender': 'male', 'status': 'single'})
japan = Node(label='country', properties={'name': 'Japan'})
redis_graph.merge("""%s-[v:visited {purpose: "work"}]->%s"""%(tom,japan))

@CLAassistant
Copy link

CLAassistant commented Dec 10, 2018

CLA assistant check
All committers have signed the CLA.

@Glitchfix
Copy link
Contributor Author

@swilly22 Can you please take a look at it and see if this is okay to merge?

"""

query = 'MERGE '
if object:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if object: I'm not sure I fully understand this condition, can you please explain this line?

@swilly22
Copy link
Contributor

@Glitchfix Can you please address my question regarding if object:

Removed check
`if object:`
@swilly22 swilly22 merged commit 8e6a2cc into RedisGraph:master Jan 21, 2019
@swilly22
Copy link
Contributor

@Glitchfix thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants