jx2lee commented on code in PR #4493:
URL: https://github.com/apache/flink-cdc/pull/4493#discussion_r3711111221
##########
flink-cdc-cli/src/main/java/org/apache/flink/cdc/cli/CliFrontendOptions.java:
##########
@@ -52,8 +52,7 @@ public class CliFrontendOptions {
.hasArg()
.desc(
"The deployment target for the execution. This can
take one of the following values "
Review Comment:
@yuxiqian Thanks for the suggestion. Since Commons CLI's Option.desc(...)
only accepts a String, we need to format Description into plain text before
passing it to the option.
Would you prefer adding a reusable PlainTextFormatter next to HtmlFormatter
under
flink-cdc-common/src/main/java/org/apache/flink/cdc/common/configuration/description/,
or keeping this scoped by adding a small private/static formatter inside
CliFrontendOptions?
--
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]
