Uh oh!
There was an error while loading. Please reload this page.
Add 'QueryJob.query_plan' property. - #3799
Conversation
tseaver
commented
Aug 12, 2017
@tswast I noted your question about creating a to-be-written This is different for |
| 'computeRatioMax': 1.09861, | ||
| 'writeRatioAvg': 3.32193, | ||
| 'writeRatioMax': 2.30258, | ||
| 'recordsRead': 100, |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
| 'writeRatioAvg': 3.32193, | ||
| 'writeRatioMax': 2.30258, | ||
| 'recordsRead': 100, | ||
| 'recordsWritten': 1, |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
tswast
commented
Aug 14, 2017
Note: I still think a class would be useful for string to int conversion, but it was only a P2 requirement from BQ team, so I won't block the PR. |
| return job | ||
| @property | ||
| def query_plan(self): |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
tseaver
commented
Sep 7, 2017
tswast
left a comment
There was a problem hiding this comment.
Thanks!
The new class will definitely be a usability improvement over using dictionaries. I agree that usage will likely be low. That's why it was a P2 item in the requirements doc from the BQ eng team.
Cherry-picked from PR #3721