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

Document local cross-repo testing for spark-rapids, spark-rapids-jni, and cudf [skip ci] #381

Merged
merged 11 commits into from
Jul 20, 2022

Conversation

gerashegalov
Copy link
Collaborator

@gerashegalov gerashegalov commented Jul 15, 2022

This PR documents how to test changes in cudf, spark-rapids-jni, and spark-rapids together locally end-to-end.

Signed-off-by: Gera Shegalov [email protected]

@gerashegalov gerashegalov self-assigned this Jul 15, 2022
@gerashegalov gerashegalov added the documentation Improvements or additions to documentation label Jul 15, 2022
@gerashegalov gerashegalov requested a review from jlowe July 15, 2022 20:55
@hyperbolic2346
Copy link
Collaborator

I like the documentation update. I'm not sure the best language is to phrase it with we though like "When we build ...". I'm not overly critical of this though and willing to live with it if it is acceptable for others. Thank you for taking the time to do this. I believe this will help new users acclimate to the process.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
@gerashegalov gerashegalov requested a review from jlowe July 19, 2022 22:28
@gerashegalov gerashegalov changed the title Document local cross-repo testing for spark-rapids, spark-rapids-jni, and cudf Document local cross-repo testing for spark-rapids, spark-rapids-jni, and cudf [skip ci] Jul 19, 2022
@gerashegalov
Copy link
Collaborator Author

build

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
Co-authored-by: @jlowe

Signed-off-by: Gera Shegalov <[email protected]>
@gerashegalov gerashegalov requested a review from jlowe July 20, 2022 18:00
@gerashegalov
Copy link
Collaborator Author

build

@gerashegalov gerashegalov merged commit c6a96d8 into NVIDIA:branch-22.08 Jul 20, 2022
@gerashegalov gerashegalov deleted the crossRepoWork branch July 20, 2022 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants