Project

General

Profile

Bug #7028

avl_destroy_nodes supports emptying, not just destroying, an avl tree

Added by Igor Kozhukhov over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Igor Kozhukhov
Category:
zfs - Zettabyte File System
Start date:
2016-06-02
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage
Gerrit CR:

Description

upstream:

commit 3106773f0e578efdfe25b9e20838cebf2293f6eb
Author: Eli Rosenthal <eli.rosenthal@delphix.com>
Date:   Fri Aug 21 14:46:00 2015 -0700
    DLPX-39393 avl_destroy_nodes supports emptying, not just destroying, an avl tree

#1

Updated by Matthew Ahrens over 4 years ago

Currently, the comment above avl_destroy_nodes only describes the use-case of destroying the tree. However, once avl_destroy_nodes removes all elements in a tree, it leaves the tree in a consistent state, allowing the same tree to be used again. The comment should be updated to indicate that this functionality is supported.

#2

Updated by Electric Monk over 4 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit 86f617e90f3e06f970c3ea9b32836e29500836ab

commit  86f617e90f3e06f970c3ea9b32836e29500836ab
Author: Eli Rosenthal <eli.rosenthal@delphix.com>
Date:   2016-06-03T15:14:45.000Z

    7028 avl_destroy_nodes supports emptying, not just destroying, an avl tree
    Reviewed by: Matthew Ahrens <mahrens@delphix.com>
    Reviewed by: Paul Dagnelie <pcd@delphix.com>
    Reviewed by: Igor Kozhukhov <ikozhukhov@gmail.com>
    Approved by: Gordon Ross <gordon.ross@nexenta.com>

#3

Updated by Josef Sipek over 4 years ago

If we really want to commit to this, the committed change isn't complete and the manpage should be updated as well.

Also available in: Atom PDF