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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
