alamb opened a new pull request #9722: URL: https://github.com/apache/arrow/pull/9722
The `sort_limit` kernel was added by @sundy-li in https://github.com/apache/arrow/pull/9602 While writing some doc examples in https://github.com/apache/arrow/pull/9721, it occured to me we could potentially simplify the API so I figured I would offer a proposed PR for comment # Rationale Since we already have a `SortOptions` structure that controls sorting options, we could also add the `limit` to that structure rather than adding a new `sort_limit` function and still avoid changing the API # Changes Move the `limit` option to `SortOptions` ---------------------------------------------------------------- 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: [email protected]
