zhuzhurk commented on code in PR #24118:
URL: https://github.com/apache/flink/pull/24118#discussion_r1472843285
##########
docs/content/docs/deployment/elastic_scaling.md:
##########
@@ -238,7 +238,7 @@ In addition, there are several related configuration
options that may need adjus
### Limitations
- **Batch jobs only**: Adaptive Batch Scheduler only supports batch jobs.
Exception will be thrown if a streaming job is submitted.
-- **BLOCKING or HYBRID jobs only**: At the moment, Adaptive Batch Scheduler
only supports jobs whose [shuffle mode]({{< ref "docs/deployment/config"
>}}#execution-batch-shuffle-mode) is `ALL_EXCHANGES_BLOCKING /
ALL_EXCHANGES_HYBRID_FULL / ALL_EXCHANGES_HYBRID_SELECTIVE`.
+- **BLOCKING or HYBRID jobs only**: At the moment, Adaptive Batch Scheduler
only supports DataStream jobs whose [shuffle mode]({{< ref
"docs/deployment/config" >}}#execution-batch-shuffle-mode) is
`ALL_EXCHANGES_BLOCKING / ALL_EXCHANGES_HYBRID_FULL /
ALL_EXCHANGES_HYBRID_SELECTIVE` and DataSet jobs whose `ExecutionMode` is
`BATCH_FORCED`.
Review Comment:
Maybe keep the original statement and add an extra statement like "Note that
for DataSet jobs which do not recognize the aforementioned shuffle mode, the
`ExecutionMode` needs to be `BATCH_FORCED` to force BLOCKING shuffle".
Later in Flink 2.0, we can directly remove the extra statement.
--
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]
