sql/inspect: remove obsolete TODO

spilchen · spilchen · commit 92f4a510c1d0 · 2025-10-02T08:48:52.000-03:00
We have no plans to add a replanner to INSPECT for topology changes. See #149460 for rationale. Removing the TODO that was previously added for it. Closes #149460 Epic: CRDB-30356 Release note: none
diff --git a/pkg/sql/inspect/inspect_job.go b/pkg/sql/inspect/inspect_job.go
@@ -60,7 +60,6 @@ func (c *inspectResumer) Resume(ctx context.Context, execCtx interface{}) error
 		return err
 	}
 
-	// TODO(149460): add a goroutine that will replan the job on topology changes
 	plan, planCtx, err := c.planInspectProcessors(ctx, jobExecCtx, pkSpans)
 	if err != nil {
 		return err

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,6 @@ func (c *inspectResumer) Resume(ctx context.Context, execCtx interface{}) error`
`60`	`60`	`return err`
`61`	`61`	`}`
`62`	`62`
`63`		`- // TODO(149460): add a goroutine that will replan the job on topology changes`
`64`	`63`	`plan, planCtx, err := c.planInspectProcessors(ctx, jobExecCtx, pkSpans)`
`65`	`64`	`if err != nil {`
`66`	`65`	`return err`