Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: fix znode subset deletion requiring parent path and recursive deletion algorithm #74

Merged
merged 1 commit into from
Aug 2, 2024
Merged

fix: fix znode subset deletion requiring parent path and recursive deletion algorithm #74

merged 1 commit into from
Aug 2, 2024

Conversation

lwpk110
Copy link
Member

@lwpk110 lwpk110 commented Aug 2, 2024

Addressed the following specific issues:

  1. Deleting a subset of znodes required the full parent path, resulting in a "zk: invalid path" error if not provided.
  2. The recursive deletion algorithm only deleted the last-level path, failing to handle intermediate-level child nodes.

@lwpk110
fix: fix znode subset deletion requiring parent path and recursive de…
90e9a3a 
…letion algorithm

Addressed the following specific issues:
1. Deleting a subset of znodes required the full parent path, resulting in a "zk: invalid path" error if not provided.
2. The recursive deletion algorithm only deleted the last-level path, failing to handle intermediate-level child nodes.
@lwpk110 lwpk110 requested a review from whg517 August 2, 2024 10:16
@lwpk110 lwpk110 merged commit 8723abb into zncdatadev:main Aug 2, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@whg517 whg517 whg517 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lwpk110 @whg517