You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, the identifier of the elasticsearch document is calculated using the nodepath.
When a node is moved, a new document with a new identifier is added to the index instead of updating the existing.
why not use the identifier instead of the node-path when finding the nodes to delete/create on move? the identifier doesn't change when moving it, right?
why not use the identifier instead of the node-path when finding the nodes to delete/create on move? the identifier doesn't change when moving it, right?
Don't remember why we use the context path for this document identifier ... sounds odd ... but some memories could be nice .... maybe in the past the node uuid change during a move operation ?
We've been running a "patched" version of this that does exactly this (use the Identifier) without a problem. I thought we did a pull request back then, but apparently not. Maybe @remuslazar can provide a pull request?
Activity
baschny commentedon Apr 5, 2021
why not use the identifier instead of the node-path when finding the nodes to delete/create on move? the identifier doesn't change when moving it, right?
dfeyer commentedon Apr 9, 2021
Don't remember why we use the context path for this document identifier ... sounds odd ... but some memories could be nice .... maybe in the past the node uuid change during a move operation ?
baschny commentedon Apr 9, 2021
We've been running a "patched" version of this that does exactly this (use the Identifier) without a problem. I thought we did a pull request back then, but apparently not. Maybe @remuslazar can provide a pull request?
remuslazar commentedon Apr 10, 2021
@baschny sure thing!
remuslazar commentedon Apr 10, 2021
@baschny @dfeyer done, see #378