alamb commented on code in PR #13727:
URL: https://github.com/apache/datafusion/pull/13727#discussion_r1880087687
##########
datafusion/core/src/datasource/avro_to_arrow/schema.rs:
##########
@@ -144,14 +148,17 @@ fn schema_to_field_with_props(
AvroSchema::Decimal(DecimalSchema {
precision, scale, ..
}) => DataType::Decimal128(*precision as u8, *scale as i8),
+ AvroSchema::BigDecimal => DataType::LargeBinary,
Review Comment:
I wondered if there would be a more useful mapping for `BigDecimal`, but it
seems like binary is good for now:
https://docs.rs/apache-avro/latest/apache_avro/schema/enum.Schema.html#variant.BigDecimal
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
