using Decimal(0) make system panicked

[waitingkuo]

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

DataFusion CLI v12.0.0
❯ select 1::decimal(0);
thread 'main' panicked at 'attempt to subtract with overflow', /Users/willy/.cargo/registry/src/github.jparrowsec.cn-1ecc6299db9ec823/arrow-23.0.0/src/datatypes/datatype.rs:1001:46
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Expected behavior
A clear and concise description of what you expected to happen.

show something like this

❯ select 10::decimal(1);
ArrowError(InvalidArgumentError("10 is too large to store in a Decimal128 of precision 1. Max is 9"))

Additional context
Add any other context about the problem here.

[HaoYang670]

This is a bug in the validate_decimal_precision in arrow-rs: https://github.com/apache/arrow-rs/blob/master/arrow-data/src/decimal.rs#L755-L756

Should we file an issue in arrow-rs?

cc @liukun4515

[alamb]

We should file a bug (and even better a fix!) upstream 👍

[liukun4515]

This is a bug in the validate_decimal_precision in arrow-rs: https://github.com/apache/arrow-rs/blob/master/arrow-data/src/decimal.rs#L755-L756

Should we file an issue in arrow-rs?

cc @liukun4515

@HaoYang670 @waitingkuo I think we should file an issue or pr to check the precision which should be greater than 0

[HaoYang670]

BTW, we could use nonZeroU8 to represent precision: https://doc.rust-lang.org/std/num/struct.NonZeroU8.html

[HaoYang670]

We can fix this bug without waiting for arrow-rs. I will file a PR.

[waitingkuo]

LGTM, thank you @HaoYang670