Merge cockroachdb#129153

129153: eval: fix hlc_to_timestamp builtin r=azhu-crl a=michae2

Builtin function `hlc_to_timestamp` calls internal function `decimalToHLC` which was assuming all input decimals had -10 as the exponent. But some decimals may have other exponents. It's more robust to call `Int64()` on the floored decimal, which handles any exponent.

Fixes: cockroachdb#129152

Release note (bug fix): Fix a bug that would cause `hlc_to_timestamp` to return an incorrect timestamp for some input decimals.

Co-authored-by: Michael Erickson <[email protected]>

pkg/sql/sem/eval/context.go

@@ -672,18 +672,19 @@ func DecimalToInexactDTimestampTZ(d *tree.DDecimal) (*tree.DTimestampTZ, error)
 }
 
 func decimalToHLC(d *tree.DDecimal) (hlc.Timestamp, error) {
-	var coef apd.BigInt
-	coef.Set(&d.Decimal.Coeff)
-	// The physical portion of the HLC is stored shifted up by 10^10, so shift
-	// it down and clear out the logical component.
-	coef.Div(&coef, big10E10)
-	if !coef.IsInt64() {
+	var floorD apd.Decimal
+	if _, err := tree.DecimalCtx.Floor(&floorD, &d.Decimal); err != nil {
+		return hlc.Timestamp{}, err
+	}
+
+	i, err := floorD.Int64()
+	if err != nil {
 		return hlc.Timestamp{}, pgerror.Newf(
 			pgcode.DatetimeFieldOverflow,
 			"timestamp value out of range: %s", d.String(),
 		)
 	}
-	return hlc.Timestamp{WallTime: coef.Int64()}, nil
+	return hlc.Timestamp{WallTime: i}, nil
 }
 
 // DecimalToInexactDTimestamp is the inverse of TimestampToDecimal. It converts

pkg/sql/sem/eval/testdata/eval/builtins

@@ -172,3 +172,13 @@ eval
 hlc_to_timestamp(1627966319623050000.0000000000)
 ----
 '2021-08-03 04:51:59.62305+00'
+
+eval
+hlc_to_timestamp(1627966319623050000)
+----
+'2021-08-03 04:51:59.62305+00'
+
+eval
+hlc_to_timestamp(1627966319623050000.00)
+----
+'2021-08-03 04:51:59.62305+00'