-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
test_sync_query_w_dml has num_dml_affected_rows unset #3742
Comments
Bah! No clue. I wish we logged the Job ID so that I could pass it on to the backend team to investigate further. |
Are the job IDs generated by the client? If yes, they probably match some specific regex with the build ID in it. |
They are in this case, but I didn't think to make them match any specific pattern. https://github.com/GoogleCloudPlatform/google-cloud-python/blob/f034c6121dabfb20cb1cf6b6ae6985b236bfef11/bigquery/google/cloud/bigquery/dbapi/cursor.py#L136 |
Nice find. Is it worth making it possible to pass in a |
It doesn't fit the DB-API, but yeah I think we need to have some way to introspect. |
Another failure: https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/2825 |
I'll update those tests to set a job ID prefix so it'll be easier to track down. |
Ah, one thing it could be is the load job may not have finished. I'll use our handy new Futures API to wait until they are done before doing the DML query. |
The other thing to keep in mind is that I'd very much like to remove the sync query method. Hopefully you got my redesign proposal doc? |
I did, but we are in fix-it week so I will not read it until Monday. |
Unfortunately the query verb doesn't take a job ID, so we'll only be able to debug half the flakes (DB-API, but not sync query). I'll finish my PR for the DB-API part, though. |
Closing, as I haven't seen this flake on the |
See: https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/2753
@tswast Any idea what would cause this "flake" to happen? (Note that the 2.7 system tests in the same build did not have this failure, as well as every other time this test case has run on CI.)
The text was updated successfully, but these errors were encountered: