Skip to content

Commit ec58dfe

Browse files
committed
add clear func
Signed-off-by: AilinKid <[email protected]>
1 parent d1c8d2f commit ec58dfe

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

pkg/planner/memo/task.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ func newTaskStack() *TaskStack {
4747
// Destroy indicates that when stack itself is useless like in the end of optimizing phase, we can destroy ourselves.
4848
func (ts *TaskStack) Destroy() {
4949
// when a taskStack itself is useless, we can destroy itself actively.
50-
ts.tasks = ts.tasks[:0]
50+
clear(ts.tasks)
5151
TaskStackPool.Put(ts)
5252
}
5353

pkg/planner/memo/task_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,4 +89,7 @@ func TestTaskFunctionality(t *testing.T) {
8989

9090
// self destroy.
9191
taskStack.Destroy()
92+
taskTaskPool = TaskStackPool.Get()
93+
require.Equal(t, len(taskTaskPool.(*TaskStack).tasks), 0)
94+
require.Equal(t, cap(taskTaskPool.(*TaskStack).tasks), 4)
9295
}

0 commit comments

Comments
 (0)