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

fix: increase batch translation timeout to 300 by telpirion · Pull Request #4357 · GoogleCloudPlatform/python-docs-samples · GitHub

fix: increase batch translation timeout to 300 - #4357

Merged
tmatsuo merged 2 commits into
masterfrom
translation-timeout
Jul 22, 2020
Merged

fix: increase batch translation timeout to 300#4357
tmatsuo merged 2 commits into
masterfrom
translation-timeout

Conversation

telpirion commented Jul 22, 2020
edited
Loading

Copy link
Copy Markdown
Contributor

This PR increases the timeout for the translate_v3_batch_translate_text_test from 240 to 300.

Fixes #4221
Fixes #4239

telpirion requested review from a team and sirtorry as code owners July 22, 2020 21:38
google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Jul 22, 2020
telpirion requested a review from tmatsuo July 22, 2020 21:38

tmatsuo 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

I think there is another variant. I thought the name was batch_translate_test_with_glossaries or sth.

Does it make sense to bump there as well?

Copy link
Copy Markdown
Contributor Author

Good call! Yes, let's bump the timeout on batch translate w/ glossary too.

tmatsuo merged commit c6fc919 into master Jul 22, 2020
tmatsuo deleted the translation-timeout branch July 22, 2020 22:59
telpirion requested a review from leahecole July 23, 2020 15:36
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
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

3 participants


Back | FazBrowse Home | New Git URL