mapleFU commented on code in PR #44867:
URL: https://github.com/apache/arrow/pull/44867#discussion_r1861848866
##########
cpp/src/arrow/array/array_dict.h:
##########
@@ -90,7 +90,11 @@ class ARROW_EXPORT DictionaryArray : public Array {
/// \param[in] type the new type object
/// \param[in] dictionary the new dictionary
/// \param[in] transpose_map transposition array of this array's indices
- /// into the target array's indices
+ /// into the target array's indices. The value of transpose_map should
+ /// be in the range [0, this->length()). And the dictionary array's
+ /// indices in the target array's indices should be in the range
+ /// of [0, dictionary->length()).
Review Comment:
It's from the user-case here https://github.com/apache/arrow/issues/44827 .
I think we should try describe this well
--
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]
