sql: use new increment when restarting sequence

Previously, if a sequence were restarted and its increment updated in
the same transaction (command), the old increment would be used to set
the value in KV.
This change corrects the `ALTER` DDL to use the new increment value.

Informs: #142914
Informs: #21564

Epic: CRDB-31283

Release note (sql change): Restarting a sequence with an updated
increment has the expected initial value.

Author: bghal

pkg/sql/alter_sequence.go

@@ -148,7 +148,7 @@ func alterSequenceImpl(
 	//
 	// The code below handles the second case.
 
-	if opts.Increment < 0 && (oldIncrement != seqDesc.SequenceOpts.Increment || oldMinValue != seqDesc.SequenceOpts.MinValue) {
+	if opts.Increment < 0 && (oldIncrement != seqDesc.SequenceOpts.Increment || seqDesc.SequenceOpts.MinValue < oldMinValue) {
 		// Only get the sequence value from KV if it's needed.
 		sequenceVal, err := getSequenceValue()
 		if err != nil {
@@ -209,13 +209,13 @@ func alterSequenceImpl(
 		}
 	}
 	if restartVal != nil {
-		// Using RESTART on a sequence should always cause the operation to run
-		// in the current transaction. This is achieved by treating the sequence
-		// as if it were just created.
-		if err := params.p.createdSequences.addCreatedSequence(seqDesc.ID); err != nil {
-			return err
+		var err error
+		if opts.Increment != 0 {
+			err = setSequenceVal(*restartVal - opts.Increment)
+		} else {
+			err = setSequenceVal(*restartVal - oldIncrement)
 		}
-		if err := params.p.SetSequenceValueByID(params.ctx, uint32(seqDesc.ID), *restartVal, false); err != nil {
+		if err != nil {
 			return err
 		}
 	}

pkg/sql/alter_table.go

@@ -11,6 +11,7 @@ import (
 	gojson "encoding/json"
 	"fmt"
 	"sort"
+	"strings"
 	"time"
 
 	"github.com/cockroachdb/cockroach/pkg/build"
@@ -1267,7 +1268,7 @@ func applyColumnMutation(
 			return err
 		}
 
-		// Alter referenced sequence for identity with sepcified option.
+		// Alter referenced sequence for identity with specified option.
 		// Does not override existing values if not specified.
 		if err := alterSequenceImpl(params, seqDesc, t.SeqOptions, t); err != nil {
 			return err
@@ -1288,7 +1289,7 @@ func applyColumnMutation(
 		if opts.Virtual {
 			optsNode = append(optsNode, tree.SequenceOption{Name: tree.SeqOptVirtual})
 		}
-		s := tree.Serialize(&optsNode)
+		s := strings.TrimSpace(tree.Serialize(&optsNode))
 		col.ColumnDesc().GeneratedAsIdentitySequenceOption = &s
 
 	case *tree.AlterTableDropIdentity:

pkg/sql/logictest/testdata/logic_test/alter_table

@@ -4415,8 +4415,8 @@ SELECT b from t_alter_identity ORDER BY b;
 1
 5
 9
-18
 20
+22
 
 statement ok
 CREATE TABLE t_identity_drop (a int GENERATED ALWAYS AS IDENTITY (START WITH 10), b int GENERATED BY DEFAULT AS IDENTITY);