vibhatha commented on code in PR #13700:
URL: https://github.com/apache/arrow/pull/13700#discussion_r936614494
##########
cpp/src/arrow/compute/exec/options.h:
##########
@@ -430,6 +430,23 @@ class ARROW_EXPORT SelectKSinkNodeOptions : public
SinkNodeOptions {
SelectKOptions select_k_options;
};
+/// \brief Make a node which selects a range of rows passed through it
+///
+/// All batches pushed to this node will be accumulated, then selected, by the
given
+/// fields. Then sorted batches will be forwarded to the generator in sorted
order and
Review Comment:
I was also thinking about `TopK` it is merely a special case or we could say
default of `Fetch`. I am bit concerned about the way I included the
`SortOptions`. If we can agree upon that change, we could replace the rest with
`Fetch`.
--
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
