zhuzhurk commented on code in PR #21626:
URL: https://github.com/apache/flink/pull/21626#discussion_r1066677286
##########
flink-core/src/main/java/org/apache/flink/configuration/JobManagerOptions.java:
##########
@@ -582,14 +578,9 @@ public enum SchedulerType {
.withDescription(
Description.builder()
.text(
- "The average size of data volume
to expect each task instance to process if %s has been set to %s. "
- + "Note that since the
parallelism of the vertices is adjusted to a power of 2, "
- + "the actual average size
will be 0.75~1.5 times this value. "
- + "It is also important to
note that when data skew occurs or the decided parallelism reaches the %s (due
to too much data), "
Review Comment:
I think this note is still needed.
Maybe with a bit improvement: "It is also important to note that" -> "Note
that"
--
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]
