Skip to content

Conversation

@HemangChothani
Copy link
Contributor

Fixes #9781

@HemangChothani HemangChothani requested a review from a team November 22, 2019 05:44
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 22, 2019
@HemangChothani HemangChothani changed the title Bigquery add create job method feat(bigquery): add create job method Nov 22, 2019
@HemangChothani HemangChothani requested a review from tswast December 4, 2019 09:55
job_config (dict): configuration job representation returned from the API.
Keyword Arguments:
source (Union[ \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These arguments are quite odd to be including in a generic "create job" function.

Better to add the needed properties to the relevant -JobConfig classes.

@HemangChothani HemangChothani added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 6, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 6, 2019
@HemangChothani
Copy link
Contributor Author

HemangChothani commented Dec 6, 2019

Unit test is failing which is unrelated to these changes. I think it's related to PR 9915.
May i create another PR to fix unit test issue?

@HemangChothani
Copy link
Contributor Author

@tswast PTAL!

@HemangChothani
Copy link
Contributor Author

Suppressed by googleapis/python-bigquery#32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants