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

Verify shuffle of decimal columns #1193

Merged
merged 2 commits into from
Dec 1, 2020
Merged

Conversation

sperlingxx
Copy link
Collaborator

This PR is about to verify shuffle of decimal columns through appending decimal column into TableBuilders for shuffle related test.

Signed-off-by: sperlingxx <lovedreamf@gmail.com>
@sperlingxx
Copy link
Collaborator Author

build

@jlowe jlowe added the test Only impacts tests label Nov 24, 2020
@sperlingxx
Copy link
Collaborator Author

build

@jlowe
Copy link
Member

jlowe commented Nov 25, 2020

Test failure appears to be related.

@sperlingxx
Copy link
Collaborator Author

Test failure appears to be related.

It failed because it relies on the contiguous split commit which has been reverted.

@jlowe
Copy link
Member

jlowe commented Nov 30, 2020

build

2 similar comments
@sperlingxx
Copy link
Collaborator Author

build

@sperlingxx
Copy link
Collaborator Author

build

@pxLi
Copy link
Collaborator

pxLi commented Dec 1, 2020

build

@sperlingxx did you upmerge your commit to the latest? It failed due to HiveStringType on 310 which jason fixed #1218 yesterday.
image

@sperlingxx
Copy link
Collaborator Author

build

1 similar comment
@sperlingxx
Copy link
Collaborator Author

build

@jlowe jlowe merged commit 84b5540 into NVIDIA:branch-0.3 Dec 1, 2020
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: sperlingxx <lovedreamf@gmail.com>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: sperlingxx <lovedreamf@gmail.com>
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
…IDIA#1193)

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.

4 participants