Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ARROW-11977: [Rust] Add documentation examples for sort kernel #9721

Closed
wants to merge 1 commit into from

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Mar 16, 2021

Rationale

Examples in the docs serve as both tests and makes the library easier to use

I figured this would be helpful the wake of the very helpful #9602 by @sundy-li

I think I am in a documenting kind of mood this week

@github-actions
Copy link

Copy link
Contributor

@sundy-li sundy-li left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alamb alamb closed this in 7865589 Mar 18, 2021
GeorgeAp pushed a commit to sirensolutions/arrow that referenced this pull request Jun 7, 2021
# Rationale
Examples in the docs serve as both tests and makes the library easier to use

I figured this would be helpful the wake of the very helpful apache#9602 by @sundy-li

I think I am in a documenting kind of mood this week

Closes apache#9721 from alamb/alamb/sort_doc_examples

Authored-by: Andrew Lamb <andrew@nerdnetworks.org>
Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
michalursa pushed a commit to michalursa/arrow that referenced this pull request Jun 13, 2021
# Rationale
Examples in the docs serve as both tests and makes the library easier to use

I figured this would be helpful the wake of the very helpful apache#9602 by @sundy-li

I think I am in a documenting kind of mood this week

Closes apache#9721 from alamb/alamb/sort_doc_examples

Authored-by: Andrew Lamb <andrew@nerdnetworks.org>
Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants