Bug #7028
avl_destroy_nodes supports emptying, not just destroying, an avl tree
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
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.
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>
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.