Improve repeat so it errors if the second argument is a non integer

[alamb]

Is your feature request related to a problem or challenge?

Prior to #14268, the following query would error:

select repeat('-1.2', 3.2);

query error DataFusion error: Error during planning: Internal error: Expect TypeSignatureClass::Native\(LogicalType\(Native\(Int64\), Int64\)\) but received Float64

After #14268 it appends 3 copies (ignores the .2 part

DataFusion CLI v44.0.0
> select repeat('-1.2', 3.2);
+-----------------------------------+
| repeat(Utf8("-1.2"),Float64(3.2)) |
+-----------------------------------+
| -1.2-1.2-1.2                      |
+-----------------------------------+
1 row(s) fetched.
Elapsed 0.023 seconds.

Describe the solution you'd like

Per @jayzhan211 the query should error if the second argument is a non intger: https://github.com/apache/datafusion/pull/14268/files#r1933031384

I still think non-integer 2nd arg should return error

Describe alternatives you've considered

No response

Additional context

No response

[jonathanc-n]

take

[alamb]

FYI @jonathanc-n it is not clear to me what should happen with #14268 (if it will be merged or not)