[ARROW-11979] [Rust] Combine limit into SortOptions

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Won't Do
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/27811
• Assignee: Andrew Lamb
• Reporter: Andrew Lamb
• Created: 16/Mar/21 11:12
• Updated: 11/Jan/23 08:23
• Resolved: 16/Mar/21 20:50

Description

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

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

1. Changes

Move the `limit` option to `SortOptions`

Issue Links

• links to: GitHub Pull Request #9722