-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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-8949] SpannerIO integration tests #11210
Conversation
retest this please |
retest this please |
Run Python PostCommit |
retest this please |
Run Python PostCommit |
Run Python 3.5 PostCommit |
retest this please |
Run Python PostCommit |
Run Python 3.5 PostCommit |
retest this please |
2 similar comments
retest this please |
retest this please |
mmmmmm |
@mszb - Do you know why the test is failing? |
@aaltay - Wordcount stream tests are failing. They are running successfully on my local machine. Jira ticket is already created for that - https://issues.apache.org/jira/browse/BEAM-9767 Apart from that, there were 3 pre-commit jobs are triggered almost the same time on Jenkins (with ~30seconds difference) and one of them is successfully completed but sadly the failed one is linked with the github. successful pre-commit: https://builds.apache.org/job/beam_PreCommit_Python_Commit/12541/ failed jobs: |
Run Python PreCommit |
OK, let's re-run to get a clear signal. |
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!
@chamikaramj: i've made some changes, could you please trigger the tests! |
Thanks. LGTM. |
Retest this please |
1 similar comment
Retest this please |
Run Python PostCommit |
3 similar comments
Run Python PostCommit |
Run Python PostCommit |
Run Python PostCommit |
Run Python 3.7 PostCommit |
2 similar comments
Run Python 3.7 PostCommit |
Run Python 3.7 PostCommit |
There were three post-commit jobs triggered for this PR, and the first one is passed. https://builds.apache.org/job/beam_PostCommit_Python37_PR/144/ (PASSED) should we try to re-run the job? |
it seems that BQ tests were broken. Can you rebase the PR, and we'll rerun the tests? |
done! |
retest this please |
retest this plesae |
Run Python 3.7 PostCommit |
1 similar comment
Run Python 3.7 PostCommit |
…n tests * wip: added spannerio read integration tests * adds spanner io write it tests * fix formatter * fix batch pipeline and it test * fix test db name * fix linting issues * fix failed batch test cases and updates changes.md * revert changes from spanner io python connector and update IT test validation code. * adds spanner it tests in the py common gradle build file Co-authored-by: Shoaib <[email protected]> Co-authored-by: Shoaib <[email protected]>
Added Spanner io integration test.
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.