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

Move input metadata tests to pyspark #165

Merged
merged 1 commit into from
Jun 15, 2020
Merged

Move input metadata tests to pyspark #165

merged 1 commit into from
Jun 15, 2020

Conversation

revans2
Copy link
Collaborator

@revans2 revans2 commented Jun 12, 2020

This fixes #91

@revans2
Copy link
Collaborator Author

revans2 commented Jun 12, 2020

build

@revans2 revans2 self-assigned this Jun 12, 2020
@revans2 revans2 added this to the Release 0.1 milestone Jun 12, 2020
@sameerz sameerz added the test Only impacts tests label Jun 15, 2020
@revans2 revans2 requested a review from abellina June 15, 2020 13:47
Copy link
Collaborator

@abellina abellina left a comment

Choose a reason for hiding this comment

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

Overall this looks good, I was wondering about the question below, as it would be nice to not repeat code if possible.

@revans2 revans2 merged commit 2c2883d into NVIDIA:branch-0.1 Jun 15, 2020
@revans2 revans2 deleted the input_int_tests branch June 15, 2020 15:22
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Only impacts tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEA] pyspark integration tests input file names
3 participants