liukun4515 commented on code in PR #2160:
URL: https://github.com/apache/arrow-rs/pull/2160#discussion_r929475763
##########
parquet/src/arrow/array_reader/builder.rs:
##########
@@ -233,6 +225,18 @@ fn build_primitive_reader(
column_desc,
arrow_type,
),
+ Some(DataType::Decimal(precision, scale)) => {
+ // read decimal data from parquet binary physical type
+ let convert =
DecimalByteArrayConvert::new(DecimalArrayConverter::new(precision as i32, scale
as i32));
Review Comment:
Yes.
previously, decimal just can be read from `int32,int64,byte_array`, but in
the definition of parquet decimal can be read from
`int32,int64,byte_array,fixed_length_byte_array`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
