Abacn commented on code in PR #32805:
URL: https://github.com/apache/beam/pull/32805#discussion_r1819191359
##########
sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery/BigQueryIO.java:
##########
@@ -3008,9 +3008,14 @@ public Write<T> withNumFileShards(int numFileShards) {
}
/**
- * Control how many parallel streams are used when using Storage API
writes. Applicable only for
- * streaming pipelines, and when {@link #withTriggeringFrequency} is also
set. To let runner
- * determine the sharding at runtime, set this to zero, or {@link
#withAutoSharding()} instead.
+ * Control how many parallel streams are used when using Storage API
writes.
+ *
+ * <p>For streaming pipelines, and when {@link #withTriggeringFrequency}
is also set. To let
+ * runner determine the sharding at runtime, set this to zero, or {@link
#withAutoSharding()}
+ * instead.
+ *
+ * <p>For batch pipelines, it inserts a redistribute. To not not reshufle
and keep the pipeline
+ * parallelism as is, set this to zero.
Review Comment:
Just confirm, if I understood correctly, `withNumStorageWriteApiStreams`
config currently is effective only in streaming pipeline (exact_once mode), but
is neglected in batch, is that correct?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
