Skip to content

Commit

Permalink
Merge #108553
Browse files Browse the repository at this point in the history
108553: changefeedccl: Correct check that job ID argument is an integer r=miretskiy a=miretskiy

Correctly check job ID is an integer.

Fixes #108500

Release note: None

Co-authored-by: Yevgeniy Miretskiy <yevgeniy@cockroachlabs.com>
  • Loading branch information
craig[bot] and Yevgeniy Miretskiy committed Aug 11, 2023
2 parents 62e59ec + 89a026c commit 165960a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/ccl/changefeedccl/alter_changefeed_stmt.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ func alterChangefeedPlanHook(
}

fn := func(ctx context.Context, _ []sql.PlanNode, resultsCh chan<- tree.Datums) error {
typedExpr, err := alterChangefeedStmt.Jobs.TypeCheck(ctx, p.SemaCtx(), types.Int)
typedExpr, err := tree.TypeCheckAndRequire(ctx, alterChangefeedStmt.Jobs, p.SemaCtx(), types.Int, "get-job-id")
if err != nil {
return err
}
Expand Down

0 comments on commit 165960a

Please sign in to comment.