[ https://issues.apache.org/jira/browse/FLINK-8135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16477332#comment-16477332 ]
ASF GitHub Bot commented on FLINK-8135: --------------------------------------- Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/5985#discussion_r188604321 --- Diff: flink-core/src/main/java/org/apache/flink/util/StringUtils.java --- @@ -348,6 +350,21 @@ public static String concatenateWithAnd(@Nullable String s1, @Nullable String s2 } } + /** + * Generates a string containing a comma-separated list of values in double-quatas. + * Uses lower-cased values returning from {@link Object#toString()} method for each element in the given array. + * Null values are spipped. + * + * @param values array of elements for the list + * + * @return The string with quated list of elements + */ + public static String toQuatedListString(Object[] values) { + return Arrays.stream(values).filter(Objects::nonNull) + .map(v -> v.toString().toLowerCase()) --- End diff -- this reminds me to check whether all enum parameters accept lower-case values... > Add description to MessageParameter > ----------------------------------- > > Key: FLINK-8135 > URL: https://issues.apache.org/jira/browse/FLINK-8135 > Project: Flink > Issue Type: Improvement > Components: Documentation, REST > Reporter: Chesnay Schepler > Assignee: Andrei > Priority: Major > Fix For: 1.5.0 > > > For documentation purposes we should add an {{getDescription()}} method to > the {{MessageParameter}} class, describing what this particular parameter is > used for and which values are accepted. -- This message was sent by Atlassian JIRA (v7.6.3#76005)