sql: internal error with BUCKET_COUNT = NULL

Previously, the `EvalShardBucketCount` function did
not properly handle the case where `BUCKET_COUNT` was
set to `NULL`, causing evaluation errors. This change
adds explicit handling for `NULL` values, ensuring that
`BUCKET_COUNT` cannot be set to `NULL` and returns an
appropriate error message.

Also added a check for NULL before sanitizing and evaluating shardBuckets
to avoid invalid operations on NULL values.
returns the error `BUCKET_COUNT cannot be NULL`.

Epic: none
Fixes: cockroachdb#130353
Release note: None

pkg/sql/catalog/tabledesc/table.go

@@ -285,6 +285,10 @@ func EvalShardBucketCount(
 		if paramVal != nil {
 			shardBuckets = paramVal
 		}
+		// Check if shardBuckets is NULL
+		if shardBuckets == tree.DNull {
+			return 0, pgerror.Newf(pgcode.InvalidParameterValue, `"BUCKET_COUNT" cannot be NULL`)
+		}
 		typedExpr, err := schemaexpr.SanitizeVarFreeExpr(
 			ctx, shardBuckets, types.Int, "BUCKET_COUNT", semaCtx, volatility.Volatile, false, /*allowAssignmentCast*/
 		)

pkg/sql/logictest/testdata/logic_test/create_table

@@ -739,6 +739,10 @@ CREATE TABLE t_bad_param (
   a INT PRIMARY KEY USING HASH WITH BUCKET_COUNT = 5 WITH (bucket_count=5)
 );
 
+statement error pq: "BUCKET_COUNT" cannot be NULL
+CREATE TABLE t_bad_param (
+  a INT PRIMARY KEY USING HASH WITH BUCKET_COUNT = NULL
+);
 
 statement error pq: invalid storage param "s2_max_level" on primary key
 CREATE TABLE t_bad_param (