Skip to content
This repository was archived by the owner on Oct 14, 2023. It is now read-only.

fix: increase batch translate timeout to 300#274

Merged
munkhuushmgl merged 1 commit intomasterfrom
translate-timeout
Aug 26, 2020
Merged

fix: increase batch translate timeout to 300#274
munkhuushmgl merged 1 commit intomasterfrom
translate-timeout

Conversation

@munkhuushmgl
Copy link
Contributor

@munkhuushmgl munkhuushmgl commented Aug 26, 2020

Fixes #267 ☕️

average time batch translate LRO to complete is between 1-3min but sometimes it takes 4-5 min to complete.

@munkhuushmgl munkhuushmgl requested a review from a team August 26, 2020 20:18
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 26, 2020
@codecov
Copy link

codecov bot commented Aug 26, 2020

Codecov Report

Merging #274 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #274   +/-   ##
=========================================
  Coverage     69.90%   69.90%           
  Complexity      220      220           
=========================================
  Files            23       23           
  Lines          1668     1668           
  Branches         46       46           
=========================================
  Hits           1166     1166           
  Misses          457      457           
  Partials         45       45           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a997d2...1685bb0. Read the comment docs.

@munkhuushmgl munkhuushmgl merged commit 75736fe into master Aug 26, 2020
@munkhuushmgl munkhuushmgl deleted the translate-timeout branch August 26, 2020 22:36
gcf-merge-on-green bot pushed a commit that referenced this pull request Oct 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

com.example.translate.BatchTranslateTextWithModelTests: testBatchTranslateTextWithModel failed

2 participants