alippai commented on a change in pull request #406:
URL: https://github.com/apache/arrow-rs/pull/406#discussion_r645975856
##########
File path: arrow/src/array/array_binary.rs
##########
@@ -728,8 +730,11 @@ impl From<ArrayData> for DecimalArray {
impl fmt::Debug for DecimalArray {
fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
write!(f, "DecimalArray<{}, {}>\n[\n", self.precision, self.scale)?;
+ let point_index = 10_i128.pow(self.scale as u32);
Review comment:
Re-used the decimal formatting. It used to be a macro, but I changed it
to an inline function. I'm not familiar with macros, but the parameters, types
and return mechanism didn't call for macro usage.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
