HuangZhenQiu commented on code in PR #26640:
URL: https://github.com/apache/flink/pull/26640#discussion_r2159506860
##########
docs/layouts/shortcodes/generated/expert_rocksdb_section.html:
##########
@@ -14,6 +14,12 @@
<td>Integer</td>
<td>The number of threads (per stateful operator) used to transfer
(download and upload) files in RocksDBStateBackend.If negative, the common (TM)
IO thread pool is used (see cluster.io-pool.size)</td>
</tr>
+ <tr>
+ <td><h5>state.backend.rocksdb.checkpoint.upload-jitter</h5></td>
+ <td style="word-wrap: break-word;">0 ms</td>
+ <td>Duration</td>
+ <td>The time interval used to create jitter for each checkpoint
file upload.</td>
Review Comment:
Thanks for these concerns. I have updated the config description to
explicitly define why and when the config should be used. It will prevent users
to mistakenly use the feature and cause side effect.
For the out of order checkpoint check, I don't think it is a concern of this
Pr. In current status, if user configures to allow multiple concurrent
checkpoint, and also there are delay on remote storage, thus cause timeout.
Whether out of order of checkpoint will happen? Whether the Flink runtime
should 100% prevent it is out of the scope of the jira and PR.
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
