[
https://issues.apache.org/jira/browse/BEAM-12260?focusedWorklogId=612183&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-612183
]
ASF GitHub Bot logged work on BEAM-12260:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 18/Jun/21 22:00
Start Date: 18/Jun/21 22:00
Worklog Time Spent: 10m
Work Description: danthev commented on a change in pull request #14713:
URL: https://github.com/apache/beam/pull/14713#discussion_r654693152
##########
File path:
sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/datastore/DatastoreV1.java
##########
@@ -225,7 +233,7 @@
* number of entities per request may be lower when we flush for the end of
a bundle or if we hit
* {@link DatastoreV1.DATASTORE_BATCH_UPDATE_BYTES_LIMIT}.
*/
- @VisibleForTesting static final int DATASTORE_BATCH_UPDATE_ENTITIES_MIN = 10;
+ @VisibleForTesting static final int DATASTORE_BATCH_UPDATE_ENTITIES_MIN = 5;
Review comment:
Somewhat similar issue, the limit of 10 was rather arbitrary and led to
some deadlines on expensive writes. Write cost outweighs the cost of processing
more RPCs, so we went down a little to accomodate that.
Both variables help with edge cases on complex entities, most traffic will
be able to go to batch size 500.
--
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: 612183)
Time Spent: 2h 10m (was: 2h)
> Java - Backport FirestoreIO connector's ramp-up to DatastoreIO connector
> ------------------------------------------------------------------------
>
> Key: BEAM-12260
> URL: https://issues.apache.org/jira/browse/BEAM-12260
> Project: Beam
> Issue Type: Improvement
> Components: io-java-gcp
> Reporter: Daniel Thevessen
> Assignee: Daniel Thevessen
> Priority: P2
> Time Spent: 2h 10m
> Remaining Estimate: 0h
>
> The FirestoreIO connector
> ([BEAM-8376|https://issues.apache.org/jira/browse/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)
