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

Cleanup QA tests #254

Merged
merged 2 commits into from
Jun 26, 2020
Merged

Cleanup QA tests #254

merged 2 commits into from
Jun 26, 2020

Conversation

revans2
Copy link
Collaborator

@revans2 revans2 commented Jun 22, 2020

This cleans up a few things with the QA tests.

It has the tests use the front door to register tables, instead of the back door.
it fixes an issue where the were trying to sort locally, but not doing it properly
removed configs that are also covered by incompat
it splits some of the select tests up between the ones that need ordering and the ones that do not.

None of these are critical changes, but I thought it would be good to do. If others disagree that is fine.

This takes over for #212

@revans2 revans2 self-assigned this Jun 22, 2020
@revans2 revans2 added the test Only impacts tests label Jun 22, 2020
@jlowe jlowe added this to the Jun 22 - Jul 2 milestone Jun 22, 2020
@revans2
Copy link
Collaborator Author

revans2 commented Jun 24, 2020

@shotai could you take a look at these changes too?

@revans2
Copy link
Collaborator Author

revans2 commented Jun 24, 2020

build

@revans2 revans2 merged commit cf2c913 into NVIDIA:branch-0.2 Jun 26, 2020
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
pxLi pushed a commit to pxLi/spark-rapids that referenced this pull request May 12, 2022
add constants file

formatting

restore some files deleted during rebase
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
Signed-off-by: spark-rapids automation <[email protected]>
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.

2 participants