emkornfield commented on a change in pull request #8475:
URL: https://github.com/apache/arrow/pull/8475#discussion_r509017441
##########
File path: cpp/src/arrow/array/validate.cc
##########
@@ -64,6 +64,13 @@ struct ValidateArrayVisitor {
return Status::OK();
}
+ Status Visit(const Decimal256Array& array) {
Review comment:
Yes, I tried to make this work, and at the moment making this seems like
it would make this change bigger then I would feel comfortable with. There are
a lot of type_traits that have confusing hierarchies (is_primitive and
is_binary_like both would include Decimal and SFINAE doesn't work out well, so
it would be an intrusive change).
----------------------------------------------------------------
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]
