pitrou commented on a change in pull request #10806:
URL: https://github.com/apache/arrow/pull/10806#discussion_r692333847
##########
File path: cpp/src/arrow/array/builder_base.h
##########
@@ -123,6 +123,14 @@ class ARROW_EXPORT ArrayBuilder {
Status AppendScalar(const Scalar& scalar, int64_t n_repeats);
Status AppendScalars(const ScalarVector& scalars);
+ /// \brief Append a range of values from an array.
+ ///
+ /// The given array must be the same type as the builder.
+ virtual Status AppendArraySliceUnchecked(const ArrayData& array, int64_t
offset,
Review comment:
We can drop "Unchecked" from the name, I think, as long as the contract
is documented.
--
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]
