kou commented on code in PR #42133: URL: https://github.com/apache/arrow/pull/42133#discussion_r1674911907
########## cpp/src/arrow/type_fwd.h: ########## @@ -89,6 +89,9 @@ using ChunkedArrayVector = std::vector<std::shared_ptr<ChunkedArray>>; using RecordBatchVector = std::vector<std::shared_ptr<RecordBatch>>; using RecordBatchIterator = Iterator<std::shared_ptr<RecordBatch>>; +template <typename TypeClass> +class TypedArrayStatistics; Review Comment: I think that the typed classes are convenient but they are required in the first version. I'll remove them. If we think that they are convenient, we can revisit this later. -- 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]
