colexec: fix decimal/interval overload error propagation

[yuzefovich]

Fixes: #57773.

Release note (bug fix): CockroachDB could previously return an internal
error when evaluating a binary expression between a Decimal and an
Interval that required a cast to a Float when the value is out of range,
and now a more user-friendly error is returned instead.

[cockroach-teamcity]

This change is 

[asubiotto]

my remaining question is why the expression in the issue doesn't get folded?

Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @asubiotto)

[yuzefovich]

I'm guessing it might have to do with the fact whether the table we're reading from is empty or not, but I'm not sure.

TFTR!

bors r+

[craig]

This PR was included in a batch that was canceled, it will be automatically retried

[craig]

Build failed (retrying...):

• GitHub CI (Cockroach)

[yuzefovich]

Yeah, I think in the case when the table is empty, we have to not perform the constant folding:

[email protected]:26257/defaultdb> CREATE TABLE t (a INT);
CREATE TABLE

Time: 2ms total (execution 2ms / network 0ms)

[email protected]:26257/defaultdb> SELECT ((-1.234E+401)::DECIMAL * '-53 years -10 mons -377 days -08:33:40.519057'::INTERVAL::INTERVAL)::INTERVAL FROM t;
  interval
------------
(0 rows)

Time: 2ms total (execution 2ms / network 0ms)

[email protected]:26257/defaultdb> INSERT INTO t VALUES (NULL);
INSERT 1

Time: 1ms total (execution 0ms / network 0ms)

[email protected]:26257/defaultdb> SELECT ((-1.234E+401)::DECIMAL * '-53 years -10 mons -377 days -08:33:40.519057'::INTERVAL::INTERVAL)::INTERVAL FROM t;
ERROR: internal error: unexpected error from the vectorized engine: strconv.ParseFloat: parsing "-1.234E+401": value out of range
SQLSTATE: XX000
DETAIL: stack trace:
github.com/cockroachdb/cockroach/pkg/sql/colexecbase/colexecerror/error.go:90: func1()
...

[craig]

Build succeeded:

• GitHub CI (Cockroach)