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

Fix a hive write test failure #10958

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

firestarman
Copy link
Collaborator

@firestarman firestarman commented Jun 3, 2024

fixes #10956

This is a bug fix for the hive write tests. In some of the tests on Spak 351, the ProjectExec will fall back to CPU due to missing the GPU version of the MapFromArrays expression.

This PR adds the ProjectExec to the allowed list of fallback for Spark 351 and the laters.

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
@firestarman
Copy link
Collaborator Author

build

@sameerz sameerz added the bug Something isn't working label Jun 3, 2024
@sameerz
Copy link
Collaborator

sameerz commented Jun 3, 2024

Minor note, does this also close #9939 ?

@firestarman firestarman closed this Jun 4, 2024
@firestarman firestarman deleted the fix-hive-error branch June 4, 2024 02:39
@firestarman
Copy link
Collaborator Author

firestarman commented Jun 4, 2024

#9939 should be fixed by #10912. Weird it was not closed automatically after merging PR #10912

@firestarman firestarman restored the fix-hive-error branch June 4, 2024 03:08
@firestarman firestarman reopened this Jun 4, 2024
@firestarman firestarman requested a review from nartal1 June 4, 2024 08:29
@firestarman firestarman merged commit 149e0d5 into NVIDIA:branch-24.08 Jun 5, 2024
87 of 88 checks passed
@firestarman firestarman deleted the fix-hive-error branch June 5, 2024 01:48
firestarman added a commit that referenced this pull request Jun 11, 2024
This is a bug fix for the hive write tests. This is simliar as #10958 but it fixes the failures for Spark 350.

The tests fail because the ProjectExec will fall back to CPU due to missing the GPU version of the MapFromArrays expression.

This PR adds the ProjectExec to the allowed list of fallback for Spark 350 and the laters.

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
SurajAralihalli pushed a commit to SurajAralihalli/spark-rapids that referenced this pull request Jul 12, 2024
This is a bug fix for the hive write tests. In some of the tests on Spak 351, 
the ProjectExec will fall back to CPU due to missing the GPU version of the MapFromArrays expression.

This PR adds the ProjectExec to the allowed list of fallback for Spark 351 and the laters.

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
SurajAralihalli pushed a commit to SurajAralihalli/spark-rapids that referenced this pull request Jul 12, 2024
This is a bug fix for the hive write tests. This is simliar as NVIDIA#10958 but it fixes the failures for Spark 350.

The tests fail because the ProjectExec will fall back to CPU due to missing the GPU version of the MapFromArrays expression.

This PR adds the ProjectExec to the allowed list of fallback for Spark 350 and the laters.

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] hive_parquet_write_test.py: test_write_compressed_parquet_into_hive_table integration test failures
3 participants