Merge #40943 #40944

40943: sql: remove unimplemented error message from COMMENT ON statement r=arulajmani a=arulajmani

Any syntax error while using the COMMENT ON statement returns an error
referencing #19472. With #19472 closed, we should no longer reference
it when the syntax is wrong.

Fixes #40713

Release note: None

Release Justification: Very minor change

40944: roachtest: tpcc-max had too low of a timeout r=jordanlewis a=jordanlewis

It's supposed to run for 6 days, but used the default timeout of 10
hours.

Closes #38558.

Release note: None
Release justification: test-only change

Co-authored-by: Arul Ajmani <arula@cockroachlabs.com>
Co-authored-by: Jordan Lewis <jordanthelewis@gmail.com>

pkg/cmd/roachtest/tpcc.go

@@ -312,6 +312,7 @@ func registerTPCC(r *testRegistry) {
 		MinVersion: maybeMinVersionForFixturesImport(cloud),
 		Tags:       []string{`weekly`},
 		Cluster:    makeClusterSpec(4, cpu(16)),
+		Timeout:    time.Duration(6*24)*time.Hour + time.Duration(10)*time.Minute,
 		Run: func(ctx context.Context, t *test, c *cluster) {
 			warehouses := 1350
 			runTPCC(ctx, t, c, tpccOptions{

pkg/sql/parser/sql.y

@@ -2166,10 +2166,6 @@ comment_stmt:
 
     $$.val = &tree.CommentOnColumn{ColumnItem: columnItem, Comment: $6.strPtr()}
   }
-| COMMENT ON error
-  {
-    return unimplementedWithIssue(sqllex, 19472)
-  }
 
 comment_text:
   SCONST