FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add a system test for 'Table.upload_from_file'. by tseaver · Pull Request #2313 · googleapis/google-cloud-python · GitHub

/ google-cloud-python Public

Add a system test for 'Table.upload_from_file'. - #2313

Merged
tseaver merged 3 commits into
googleapis:masterfrom
tseaver:2133-bigquery-systest-table-upload_from_file
Sep 15, 2016
ConversationCommits (3)ChecksFiles changed
Merged

Add a system test for 'Table.upload_from_file'.#2313
tseaver merged 3 commits into
googleapis:masterfrom
tseaver:2133-bigquery-systest-table-upload_from_file

Conversation

tseaver commented Sep 13, 2016

Copy link
Copy Markdown
Contributor

Closes #2133.

/cc @jonparrott, @matheuspatury

I will cancel the PR job on Travis, as this PR changes only system tests.

tseaver added testing api: bigquery Issues related to the BigQuery API. labels Sep 13, 2016
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 13, 2016

tseaver commented Sep 13, 2016

Copy link
Copy Markdown
Contributor Author

@jgeewax, @omaray the new system test passes, and shows correct usage of Table.upload_from_file.

dhermes commented Sep 13, 2016

Copy link
Copy Markdown
Contributor

Before reviewing, this doesn't seem to fix the title of the issue:

bigquery.table.Table.upload_from_file fails with 400

Should the issue be renamed to be more accurate?

dhermes reviewed Sep 13, 2016
View reviewed changes
Comment thread system_tests/bigquery.py
writer.writerows(ROWS)
csv_file.flush()

with open(csv_file.name, 'rb') as csv_read:

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

tseaver commented Sep 14, 2016

Copy link
Copy Markdown
Contributor Author

Before reviewing, this doesn't seem to fix the title of the issue: "bigquery.table.Table.upload_from_file fails with 400"

The OP in that issue made an understandable error, calling job.begin() on the job returned from table.upload_from_file(): the issue is that the job already started, and the resource returned for the job does not contain enough information to (re)start it. This PR just demonstrates that table.upload_from_file() works as expected without making that call.

Should the issue be renamed to be more accurate?

We could rename the issue, or just close it and move on.

dhermes commented Sep 14, 2016

Copy link
Copy Markdown
Contributor

We could rename the issue, or just close it and move on.

Can you rename it to be more accurate? This way if anyone comes across it again they won't be misled.

tseaver commented Sep 14, 2016

Copy link
Copy Markdown
Contributor Author

Renamed.

tseaver mentioned this pull request Sep 14, 2016
Closed

dhermes commented Sep 15, 2016

Copy link
Copy Markdown
Contributor

LGTM

tseaver commented Sep 15, 2016

Copy link
Copy Markdown
Contributor Author

@dhermes Any remaining issues?

dhermes approved these changes Sep 15, 2016
View reviewed changes

dhermes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

tseaver merged commit d861610 into googleapis:master Sep 15, 2016
tseaver deleted the 2133-bigquery-systest-table-upload_from_file branch September 15, 2016 16:13
dhermes mentioned this pull request Sep 19, 2016
Merged
parthea pushed a commit that referenced this pull request Sep 22, 2023
[video] Streaming AutoML Classification [(#2313)](GoogleCloudPlatform…
d4efaaa 
…/python-docs-samples#2313)

* Video Intelligence Beta - Streaming/Live Streaming support for AutoML custom models

* add test skeleton

* skeleton

* more skeleton code

* update sample: update video codec/test/model_id/etc.

* lint

* mask project id

* Noah's and Rebecca's suggestions
parthea added a commit that referenced this pull request Nov 24, 2025
tests: remove generated routing parameter test for bidi/client side s…
332a50f 
…treaming (#2313)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
dhermes dhermes approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: bigquery Issues related to the BigQuery API. cla: yes This human has signed the Contributor License Agreement. testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL