[GitHub] [iceberg] rdblue commented on issue #7420: DecimalType declaration check missing proper assertions

["rdblue (via GitHub)" <gi...@apache.org>]

rdblue commented on issue #7420:
URL: https://github.com/apache/iceberg/issues/7420#issuecomment-1526522715

   It's reasonable for precision=0 to be disallowed, but there are some cases where precision can be less than scale. Scale is where the decimal point goes, so you can have `decimal(3,2)` like `3.14` or `decimal(3,4)` like `0.0314`. Scale can also be negative to encode small precision large numbers, like `decimal(3, -4)` and `3140000`.
   
   We should go with the SQL spec, but it may be that these are valid even if Trino doesn't support them.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org