alamb commented on a change in pull request #1223:
URL: https://github.com/apache/arrow-rs/pull/1223#discussion_r795166839
##########
File path: arrow/src/datatypes/datatype.rs
##########
@@ -189,6 +194,12 @@ impl fmt::Display for DataType {
}
}
+/// The maximum precision for [DataType::Decimal] values
+pub const DECIMAL_MAX_PRECISION: usize = 38;
Review comment:
I agree that we should remove the duplication in DataFusion -- when this
PR is merged, I can file a ticket in DataFusion to upgrade to the new
DecimalAPI and remove the duplication
--
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...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
