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

Temporarily xfail failing test_iceberg_parquet_read_round_trip test #6756

Merged
merged 1 commit into from
Oct 11, 2022

Conversation

tgravescs
Copy link
Collaborator

related to #6718

xfail so we get the ci pipeline running the rest of the way.

Signed-off-by: Thomas Graves [email protected]

@tgravescs tgravescs added the test Only impacts tests label Oct 11, 2022
@tgravescs tgravescs added this to the Oct 10 - Oct 21 milestone Oct 11, 2022
@tgravescs tgravescs self-assigned this Oct 11, 2022
@tgravescs
Copy link
Collaborator Author

build

@tgravescs tgravescs changed the title Temporarily xfail failing iceberg test Temporarily xfail failing test_iceberg_parquet_read_round_trip test Oct 11, 2022
Copy link
Contributor

@jbrennan333 jbrennan333 left a comment

Choose a reason for hiding this comment

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

+1 lgtm

@tgravescs tgravescs merged commit 2e8427b into NVIDIA:branch-22.12 Oct 11, 2022
@tgravescs tgravescs deleted the icexfail branch October 11, 2022 17:54
tgravescs added a commit that referenced this pull request Oct 13, 2022
tgravescs added a commit that referenced this pull request Oct 13, 2022
…p test" (#6783)

* Revert "Temporarily xfail failing iceberg test (#6756)"

This reverts commit 2e8427b.

* signoff

Signed-off-by: Thomas Graves <[email protected]>

Signed-off-by: Thomas Graves <[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