curator icon indicating copy to clipboard operation
curator copied to clipboard

[CURATOR-373] Add option to PersistentNode to not overwrite/delete existing node

Open jira-importer opened this issue 9 years ago • 2 comments

Currently, when PersistentNode.start() is called, and a node at the specified path already exists, this existing node's data is overwritten with the PersistentNode's data.

Similarly, when PersistentNode.close() is called, the node is deleted, even if a node existed at the specified path before PersistentNode.start() was called.

Add an option to PersistentNode to not overwrite/delete an existing node.


Originally reported by lakospeter, imported from: Add option to PersistentNode to not overwrite/delete existing node
  • status: Open
  • priority: Major
  • resolution: Unresolved
  • imported: 2025-01-21

jira-importer avatar Jan 02 '17 16:01 jira-importer

githubbot:

GitHub user lakospeter91 opened a pull request:

https://github.com/apache/curator/pull/182

CURATOR-373 Add option to PersistentNode to not overwrite/delete existing node

https://issues.apache.org/jira/browse/CURATOR-373

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ustream/curator CURATOR-373

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/curator/pull/182.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #182


commit b229a6fd33a79e5280f014dacedb2f1743fb4619
Author: Lakos Péter
Date: 2017-01-02T16:41:56Z

CURATOR-373 Add option to PersistentNode to not overwrite/delete existing node


jira-importer avatar Jan 02 '17 16:01 jira-importer

githubbot:

Github user Randgalt commented on the issue:

https://github.com/apache/curator/pull/182

If I understand this PR correctly, overwriteExisting only applies on the first use, right? If so, I'd like to see this renamed to something that better reflects it's use. `overwriteInitialExisting` or `overwriteStartingExisting` or something.

jira-importer avatar Jan 08 '17 16:01 jira-importer