kou commented on code in PR #46779: URL: https://github.com/apache/arrow/pull/46779#discussion_r2153179461
########## cpp/src/arrow/chunked_array.h: ########## @@ -160,12 +160,22 @@ class ARROW_EXPORT ChunkedArray { /// /// Two chunked arrays can be equal only if they have equal datatypes. /// However, they may be equal even if they have different chunkings. + /// + /// Setting \ref arrow::EqualOptions::use_atol to true is equivalent to + /// using \ref arrow::ChunkedArray::ApproxEquals Review Comment: ```suggestion /// using \ref arrow::ChunkedArray::ApproxEquals. ``` ########## cpp/src/arrow/record_batch.h: ########## @@ -120,6 +120,10 @@ class ARROW_EXPORT RecordBatch { /// \brief Determine if two record batches are exactly equal /// + /// Setting \ref arrow::EqualOptions::use_atol to true is equivalent to + /// using \ref arrow::RecordBatch::ApproxEquals Although, Schema comparison is included, Review Comment: ```suggestion /// using \ref arrow::RecordBatch::ApproxEquals. Although, schema comparison is included, ``` ########## cpp/src/arrow/chunked_array.h: ########## @@ -160,12 +160,22 @@ class ARROW_EXPORT ChunkedArray { /// /// Two chunked arrays can be equal only if they have equal datatypes. /// However, they may be equal even if they have different chunkings. + /// + /// Setting \ref arrow::EqualOptions::use_atol to true is equivalent to + /// using \ref arrow::ChunkedArray::ApproxEquals bool Equals(const ChunkedArray& other, const EqualOptions& opts = EqualOptions::Defaults()) const; + /// \brief Determine if two chunked arrays are equal. + /// + /// Setting \ref arrow::EqualOptions::use_atol to true is equivalent to + /// using \ref arrow::ChunkedArray::ApproxEquals Review Comment: ```suggestion /// using \ref arrow::ChunkedArray::ApproxEquals. ``` -- 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