Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

docs: Updatesquery toquery and wait in samples/desktopapp/user_credentials.py#1787

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
kiraksi merged 8 commits intogoogleapis:mainfromDevStephanie:Stephanie4
Jan 22, 2024

Conversation

@DevStephanie
Copy link
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as abug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@DevStephanieDevStephanie requested review froma team ascode ownersJanuary 17, 2024 04:36
@product-auto-labelproduct-auto-labelbot added size: xsPull request size is extra small. api: bigqueryIssues related to the googleapis/python-bigquery API. samplesIssues that are directly related to samples. labelsJan 17, 2024
@kiraksikiraksi added kokoro:force-runAdd this label to force Kokoro to re-run the tests. owlbot:runAdd this label to trigger the Owlbot post processor. labelsJan 17, 2024
@gcf-owl-botgcf-owl-botbot removed the owlbot:runAdd this label to trigger the Owlbot post processor. labelJan 17, 2024
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-runAdd this label to force Kokoro to re-run the tests. labelJan 17, 2024
@kiraksikiraksi requested review fromtswast and removed request form-strzelczyk andnayaknishantJanuary 17, 2024 11:14
@kiraksikiraksi added the kokoro:force-runAdd this label to force Kokoro to re-run the tests. labelJan 17, 2024
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-runAdd this label to force Kokoro to re-run the tests. labelJan 17, 2024
@kiraksi
Copy link
Contributor

Looks like you forgot to remove query_job.result() here as I believe query_and_wait returns the result itself, as the cause of your sample errors. Also Tim mentioned in my PR that he wanted to query_job to be renamed to result as well which may also be helpful to you here!

Copy link
Contributor

@tswasttswast left a comment

Choose a reason for hiding this comment

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

Thanks!

@kiraksikiraksi added the kokoro:force-runAdd this label to force Kokoro to re-run the tests. labelJan 18, 2024
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-runAdd this label to force Kokoro to re-run the tests. labelJan 18, 2024
@kiraksikiraksi added the kokoro:force-runAdd this label to force Kokoro to re-run the tests. labelJan 18, 2024
@kiraksikiraksienabled auto-merge (squash)January 18, 2024 22:15
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-runAdd this label to force Kokoro to re-run the tests. labelJan 18, 2024
@chalmerlowechalmerlowe added the type: docsImprovement to the documentation for an API. labelJan 22, 2024
@kiraksikiraksi added the cla: yesThis human has signed the Contributor License Agreement. labelJan 22, 2024
@kiraksikiraksi added kokoro:force-runAdd this label to force Kokoro to re-run the tests. and removed cla: yesThis human has signed the Contributor License Agreement. labelsJan 22, 2024
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-runAdd this label to force Kokoro to re-run the tests. labelJan 22, 2024
@kiraksikiraksi merged commit89f1299 intogoogleapis:mainJan 22, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@tswasttswasttswast approved these changes

Assignees

No one assigned

Labels

api: bigqueryIssues related to the googleapis/python-bigquery API.samplesIssues that are directly related to samples.size: xsPull request size is extra small.type: docsImprovement to the documentation for an API.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@DevStephanie@kiraksi@tswast@chalmerlowe@yoshi-kokoro

[8]ページ先頭

©2009-2025 Movatter.jp