bkietz commented on a change in pull request #8894:
URL: https://github.com/apache/arrow/pull/8894#discussion_r552843114



##########
File path: cpp/src/arrow/dataset/partition.h
##########
@@ -294,5 +285,22 @@ class ARROW_DS_EXPORT PartitioningOrFactory {
   std::shared_ptr<Partitioning> partitioning_;
 };
 
+/// \brief Assemble lists of indices of identical rows.
+///
+/// \param[in] by A StructArray whose columns will be used as grouping 
criteria.
+/// \return A StructArray mapping unique rows (in field "values", represented 
as a
+///         StructArray with the same fields as `by`) to lists of indices where
+///         that row appears (in field "groupings").

Review comment:
       I'll address this in ARROW-10247




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to