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
[BEAM-8889] Upgrades gcsio to 2.1.3 #11651
Conversation
Retest this please |
Run Java PostCommit |
Run Dataflow ValidatesRunner |
No performance regression with this change. (Tested with internal tests) |
Run Java PreCommit |
Run Java PostCommit |
It appears that it actually has a problem with dependency. Java PostCommit / testReadEmptySketchFromBigQuery
Java PostCommit / SpannerWriteIT/testWrite_2
|
This is most likely caused by gcsio 2.1.3 dependency to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
It turns out that |
Retest this please |
2 similar comments
Retest this please |
Retest this please |
Run Python2_PVR_Flink PreCommit |
1 similar comment
Run Python2_PVR_Flink PreCommit |
Have you run the linkage checker ? |
This is the result of linkage chekcer. (it was challenging because it seems to require Java8 strictly to finish it smoothly)
|
Run Dataflow ValidatesRunner |
1 similar comment
Run Dataflow ValidatesRunner |
Having trouble triggering tests at the moment. I think this is good to go when we get following to pass with the latest code. |
LGTM |
Run Java PostCommit |
Run Java PostCommit |
1 similar comment
Run Java PostCommit |
Run Dataflow ValidatesRunner |
Run Java PostCommit |
whoops |
It looks like Dataflow VR tests are going to fail:
|
Thanks for running the test and the result doesn't make sense because this change won't introduce any runtime behavior other than dependency. Is it possible to be caused by flaky environment? Can we rerun the test again? |
Run Dataflow ValidatesRunner |
Thanks, Brian and Chamikara! |
This is part of BEAM-8889 to get new GCS connector 2.1.3 after #11315. This is the first version to support gRPC protocol so it has a new dependency against gRPC and it affects the dependency tree of beam SDK transitively. gRPC feature is only available with
use_grpc_for_gcs
experiment flag and the whitelisted GCS buckets.Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username
).[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.CHANGES.md
with noteworthy changes.See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)
Pre-Commit Tests Status (on master branch)
See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.