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

Add README for the compute module. #1940

merged 1 commit into from Jun 24, 2022


Copy link

Signed-off-by: remzi

Which issue does this PR close?

Closes #1533 .

Rationale for this change

Improve the readability of the code.

What changes are included in this PR?

The README is mostly copied from the c++ repo:

Something different is that the term function and kernel in c++ repo are not the same as that in rust repo. (I guess we don't implement those two structs in rust.)

I defined kernel as a general operation (such as compare, sort, take ...) which contains many functions.

Are there any user-facing changes?

NO, I guess.

Signed-off-by: remzi <>
@github-actions github-actions bot added the arrow Changes to the arrow crate label Jun 24, 2022
@alamb alamb merged commit c5cf2f2 into apache:master Jun 24, 2022
16 of 17 checks passed
@alamb alamb added the documentation Improvements or additions to documentation label Jun 24, 2022
Copy link

alamb commented Jun 24, 2022

Thanks @HaoYang670

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
arrow Changes to the arrow crate documentation Improvements or additions to documentation
None yet

Successfully merging this pull request may close these issues.

Add README for the compute kernels
2 participants