return state instead of schedule (#55942)

pingcap · Sep 9, 2024 · 499e45f · 499e45f
1 parent 2d1ed88
commit 499e45f
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/pkg/executor/show_placement.go b/pkg/executor/show_placement.go
@@ -503,13 +503,9 @@ func fetchPartitionScheduleState(ctx context.Context, scheduleState map[int64]in
 }
 
 func fetchTablePartitionScheduleState(ctx context.Context, scheduleState map[int64]infosync.PlacementScheduleState, table *model.TableInfo, state infosync.PlacementScheduleState) (infosync.PlacementScheduleState, error) {
-	var (
-		schedule infosync.PlacementScheduleState
-		err      error
-	)
 	if table.GetPartitionInfo() != nil {
 		for _, part := range table.GetPartitionInfo().Definitions {
-			schedule, err = fetchScheduleState(ctx, scheduleState, part.ID)
+			schedule, err := fetchScheduleState(ctx, scheduleState, part.ID)
 			if err != nil {
 				return infosync.PlacementScheduleStatePending, err
 			}
@@ -520,7 +516,7 @@ func fetchTablePartitionScheduleState(ctx context.Context, scheduleState map[int
 		}
 	}
 
-	return schedule, nil
+	return state, nil
 }
 
 func fetchTableScheduleState(ctx context.Context, scheduleState map[int64]infosync.PlacementScheduleState, table *model.TableInfo) (infosync.PlacementScheduleState, error) {