[ 
https://issues.apache.org/jira/browse/BEAM-10597?focusedWorklogId=472146&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-472146
 ]

ASF GitHub Bot logged work on BEAM-10597:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 18/Aug/20 21:52
            Start Date: 18/Aug/20 21:52
    Worklog Time Spent: 10m 
      Work Description: robinyqiu commented on a change in pull request #12403:
URL: https://github.com/apache/beam/pull/12403#discussion_r472514238



##########
File path: 
sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery/BigQueryServicesImpl.java
##########
@@ -867,6 +872,7 @@ public void deleteDataset(String projectId, String 
datasetId)
         }
         try {
           sleeper.sleep(nextBackoffMillis);
+          throttlingMilliSeconds.inc(nextBackoffMillis);

Review comment:
       > Yes, we should use backoff1. Rate limit errors only reach to this 
point after 2 minutes of backoffs by backoff1 silently inside of the future.
   
   I see. That make sense to me.
   
   > Why do you think it's over-calculated?
   
   Because I am not sure how this metrics is used downstream. I vaguely 
remember Dataflow autoscaling will use this number divided by the total time 
spent on work item to yield a fraction which signals throttling. If the total 
time is not accumulating time spent on all threads then we may over-calculate.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 472146)
    Time Spent: 2h 20m  (was: 2h 10m)

> Propagate BigQuery streaming insert throttled time to Dataflow worker
> ---------------------------------------------------------------------
>
>                 Key: BEAM-10597
>                 URL: https://issues.apache.org/jira/browse/BEAM-10597
>             Project: Beam
>          Issue Type: New Feature
>          Components: io-java-gcp
>            Reporter: Robin Qiu
>            Assignee: Robin Qiu
>            Priority: P2
>          Time Spent: 2h 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to