Hey team, I see that support for storing decimals ...
# generaln
Nizar Hejazi
03/23/2022, 1:33 AMHey team, I see that support for storing decimals as byte[] and SumPrecision function was added in Pinot 0.6.0. We use Pinot through Presto\Trino. I think byte[] type will be mapped to Presto’s varbinary. I cannot find a builtin Presto function that is equivalent to Pinot’s bytesToBigDecimal? May need to define a Presto UDF.
Do you plan to look into adding support for inferring Presto decimal type from Pinot byte[] (if Pinot can indicate somehow that byte[] stores a decimal to Presto) similar to
pinot.infer-date-type-in-schemapinot.infer-timestamp-type-in-schemaNizar Hejazi
03/26/2022, 12:15 AM@User @User @User Created an issue to track the ask: https://github.com/apache/pinot/issues/8418. Please review and let me know your feedback.
Nizar Hejazi
03/28/2022, 3:53 PM@User @User I will go ahead with doing the change
👍 1
Nizar Hejazi
04/05/2022, 1:46 PM@User @User Sent PR to support single-valued BigDecimal columns in most SQL statements and Pinot transforms/operators: https://github.com/apache/pinot/pull/8468. Can you please take a look? thanks.
👍 1
Nizar Hejazi
04/11/2022, 7:35 AM@User @User As discussed, I broke my PR into multiple PRs. Here is the first PR (please review):
https://github.com/apache/pinot/pull/8503