-
Notifications
You must be signed in to change notification settings - Fork 69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue #542: Add a strategy to merge edge nodes properties when creating relationships using Relationship pattern strategy #544
Conversation
6d345ed
to
3de0487
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a very nit pick! Thank you so much @mroiter-larus !
@@ -31,4 +31,3 @@ Thumbs.db | |||
bin | |||
doc/node | |||
doc/node_modules | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please restore it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm having second thoughts on this. I want to change the naming convention and split the properties for nodes and relationships.
@@ -71,6 +74,8 @@ class Neo4jSinkConnectorConfig(originals: Map<*, *>): Neo4jConnectorConfig(confi | |||
const val TOPIC_CDC_SCHEMA = "neo4j.topic.cdc.schema" | |||
const val TOPIC_CUD = "neo4j.topic.cud" | |||
|
|||
const val MERGE_PROPERTIES_ENABLED = "neo4j.merge.properties.enabled" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const val MERGE_PROPERTIES_ENABLED = "neo4j.merge.properties.enabled" | |
const val TOPIC_PATTERN_MERGE_NODE_PROPERTIES_ENABLED = "neo4j.topic.pattern.merge.node.properties.enabled" | |
const val TOPIC_PATTERN_MERGE_RELATIONSHIP_PROPERTIES_ENABLED = "neo4j.topic.pattern.merge.relationship.properties.enabled" |
And please adapt all accordingly.
3de0487
to
03fffe8
Compare
…s when creating relationships using Relationship pattern strategy
03fffe8
to
fb4dfd5
Compare
0e9bf7c
to
f02042b
Compare
Fixes #542
Proposed Changes (Mandatory)
neo4j.merge.properties.enabled
: If enabled edge nodes properties will be merged when creating relationships using sinkRelationship pattern strategy
(default false: edge node properties will be overwritten).