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

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

                Author: ASF GitHub Bot
            Created on: 02/Jun/21 03:28
            Start Date: 02/Jun/21 03:28
    Worklog Time Spent: 10m 
      Work Description: chamikaramj commented on a change in pull request 
#14723:
URL: https://github.com/apache/beam/pull/14723#discussion_r643625554



##########
File path: sdks/python/apache_beam/io/gcp/datastore/v1new/datastoreio.py
##########
@@ -276,15 +277,33 @@ class _Mutate(PTransform):
   Only idempotent Datastore mutation operations (upsert and delete) are
   supported, as the commits are retried when failures occur.
   """
-  def __init__(self, mutate_fn):
+
+  # Default hint for the expected number of workers in the ramp-up throttling
+  # step for write or delete operations.
+  _DEFAULT_HINT_NUM_WORKERS = 500

Review comment:
       cumulativeThrottlingSeconds results in 
"total_throttler_wait_time_seconds" property of the Dataflow API being set 
which results in reporting the per-worker throttled number of seconds to the 
Dataflow service. (This part of the code, Dataflow Python legacy worker, is not 
open source).
   




-- 
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: 604916)
    Time Spent: 5h 10m  (was: 5h)

> Python - Backport FirestoreIO connector's ramp-up to DatastoreIO connector
> --------------------------------------------------------------------------
>
>                 Key: BEAM-12272
>                 URL: https://issues.apache.org/jira/browse/BEAM-12272
>             Project: Beam
>          Issue Type: Improvement
>          Components: io-py-gcp
>            Reporter: Daniel Thevessen
>            Assignee: Daniel Thevessen
>            Priority: P2
>          Time Spent: 5h 10m
>  Remaining Estimate: 0h
>
> The FirestoreIO connector (BEAM-8376) for the Java SDK is currently out for 
> review.  
> Some of the features for the Firestore connector are new relative to the 
> Datastore connector, so the Firestore team would like to add them there as 
> well. Most notably the Firestore connector has a gradual ramp-up feature that 
> slowly increases throughput in line with Datastore/Firestore best practices.  
> Some minor adjustments to constants for batch sizing and backoff may also be 
> worth looking at.



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

Reply via email to