davidradl commented on code in PR #26521:
URL: https://github.com/apache/flink/pull/26521#discussion_r2070431630
##########
flink-libraries/flink-state-processing-api/src/main/java/org/apache/flink/state/table/SavepointConnectorOptions.java:
##########
@@ -132,8 +133,27 @@ public enum StateType {
.stringType()
.noDefaultValue()
.withDescription(
- "Defines the format class scheme for decoding
value data. "
- + "When it's not provided then it tries to
be inferred from the SQL type (only primitive types supported).");
+ Description.builder()
+ .text(
+ "Defines the format class scheme
for decoding value data. "
+ + "Either %s or %s can be
specified. "
+ + "When none of them
provided then it tries to be inferred from the SQL type (only primitive types
supported).",
+ code(VALUE_FORMAT),
code(VALUE_TYPE_INFO_FACTORY))
+ .build());
+
+ /** Placeholder {@link ConfigOption}. Not used for retrieving values. */
+ public static final ConfigOption<String>
VALUE_TYPE_INFO_FACTORY_PLACEHOLDER =
+ ConfigOptions.key(String.format("%s.#.%s", FIELDS,
VALUE_TYPE_INFO_FACTORY))
+ .stringType()
+ .noDefaultValue()
+ .withDescription(
+ Description.builder()
+ .text(
+ "Defines the type information
factory for decoding value data. "
Review Comment:
I suggest similar wording to the above.
--
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
