tillrohrmann commented on a change in pull request #18083:
URL: https://github.com/apache/flink/pull/18083#discussion_r785025300
##########
File path:
flink-core/src/main/java/org/apache/flink/configuration/CheckpointingOptions.java
##########
@@ -164,9 +164,14 @@
ConfigOptions.key("taskmanager.state.local.root-dirs")
.noDefaultValue()
.withDescription(
- "The config parameter defining the root
directories for storing file-based state for local "
- + "recovery. Local recovery currently only
covers keyed state backends. Currently, MemoryStateBackend does "
- + "not support local recovery and ignore
this option");
+ Description.builder()
Review comment:
My reasoning was that I wanted to give users still the option to
configure a different local state directory. Especially if they want to
configure multiple volumes, this is the only way now.
--
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]
