Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Disable 340 for premerge and nightly #5689

Merged
merged 1 commit into from
May 30, 2022

Conversation

firestarman
Copy link
Collaborator

@firestarman firestarman commented May 30, 2022

Spark 340 has removed the AnsiCast by apache/spark@5f0bae7, which will probably lead to much work in plugin.
So disable 340 for premerge and nightly temporarily. But need to add it back when the issue #5688 is fixed.

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
@pxLi pxLi added the build Related to CI / CD or cleanly building label May 30, 2022
Copy link
Member

@pxLi pxLi left a comment

Choose a reason for hiding this comment

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

+1 for CI

Copy link
Collaborator

@res-life res-life left a comment

Choose a reason for hiding this comment

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

LGTM

@firestarman
Copy link
Collaborator Author

build

@firestarman firestarman merged commit 4769926 into NVIDIA:branch-22.08 May 30, 2022
@firestarman firestarman deleted the ci-disable-340 branch May 30, 2022 05:36
firestarman added a commit to firestarman/spark-rapids that referenced this pull request Jun 2, 2022
HaoYang670 pushed a commit to HaoYang670/spark-rapids that referenced this pull request Jun 6, 2022
Signed-off-by: Firestarman <firestarmanllc@gmail.com>
firestarman added a commit to firestarman/spark-rapids that referenced this pull request Jun 6, 2022
tgravescs pushed a commit that referenced this pull request Jun 13, 2022
* Remove AnsiCast for 340

Signed-off-by: Firestarman <firestarmanllc@gmail.com>

* Revert "Disable 340 for premerge and nightly (#5689)"

This reverts commit 4769926.

* Disable the failing tests

Signed-off-by: Firestarman <firestarmanllc@gmail.com>

* Fix a new build error about parquet column

Signed-off-by: Firestarman <firestarmanllc@gmail.com>

* Add comment for the disabled tests

Signed-off-by: Firestarman <firestarmanllc@gmail.com>

* address comments

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants