-
Notifications
You must be signed in to change notification settings - Fork 6.5k
data-science-onramp.data-cleaning.clean_test: test_clean failed #5312
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
Comments
Looks like this issue is flaky. 😟 I'm going to leave this open and stop commenting. A human should fix and close this. When run at the same commit (c3a6599), this test passed in one build (Build Status, Sponge) and failed in another build (Build Status, Sponge). |
@bradmiro do you have bandwidth to take a look at this? I'm wondering if it's happening because of the updated pyarrow dependency |
It's OOSLO on our dashboard, I'll try to fix this. |
It doesn't seem that this error is deterministic. |
It's not clear what was the problem from the current logs. It failed in this line:
Maybe we can add debugging output before this call. |
fixes GoogleCloudPlatform#5312 It won't fix the issue, but will allow us to debug better.
This failure seems very rare. I don't see same failure in days in our periodic builds. I'm adding some debug output with #5343. |
@leahecole @tmatsuo apologies, this completely slipped through my inbox. I'll take a look at this later today. PyArrow dependencies are pinned in some older versions of Dataproc, perhaps we should update to Dataproc 2.0 in our samples. I will double check compatibility. |
fixes #5312 It won't fix the issue, but will allow us to debug better.
This test failed!
To configure my behavior, see the Flaky Bot documentation.
If I'm commenting on this issue too often, add the
flakybot: quiet
label andI will stop commenting.
commit: c3a6599
buildURL: Build Status, Sponge
status: failed
Test output
The text was updated successfully, but these errors were encountered: