dongjoon-hyun commented on code in PR #42801: URL: https://github.com/apache/spark/pull/42801#discussion_r1315213044
########## connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/functions.scala: ########## @@ -4284,12 +4285,22 @@ object functions { * prints '+' for positive values but 'MI' prints a space.</li> <li>'PR': Only allowed at the * end of the format string; specifies that the result string will be wrapped by angle * brackets if the input value is negative.</li> </ul> + * If `e` is a datetime, `format` shall be a valid datetime pattern, see + * <a href="https://spark.apache.org/docs/latest/sql-ref-datetime-pattern.html">Datetime Patterns</a>. + * If `e` is a binary, it is converted to a string in one of the formats: + * <ul> + * <li>'base64': a base 64 string.</li> + * <li>'hex': a string in the hexadecimal format.</li> + * <li>'utf-8': the input binary is decoded to UTF-8 string.</li> + * </ul> Review Comment: To @MaxGekk , it seems our Linter CI doesn't allow this kind of human-friendly formatting in the comment. - https://github.com/MaxGekk/spark/actions/runs/6076537960/job/16484751098 ``` The scalafmt check failed on connector/connect at following occurrences: Requires formatting: functions.scala Before submitting your change, please make sure to format your code using the following command: ./build/mvn -Pscala-2.12 scalafmt:format -Dscalafmt.skip=false -Dscalafmt.validateOnly=false -Dscalafmt.changedOnly=false -pl connector/connect/common -pl connector/connect/server -pl connector/connect/client/jvm Error: Process completed with exit code 1. ``` The recommended style is something like the following. Please fix the style by run the above command line. ``` - * brackets if the input value is negative.</li> </ul> - * If `e` is a datetime, `format` shall be a valid datetime pattern, see - * <a href="https://spark.apache.org/docs/latest/sql-ref-datetime-pattern.html">Datetime Patterns</a>. - * If `e` is a binary, it is converted to a string in one of the formats: - * <ul> - * <li>'base64': a base 64 string.</li> - * <li>'hex': a string in the hexadecimal format.</li> - * <li>'utf-8': the input binary is decoded to UTF-8 string.</li> - * </ul> + * brackets if the input value is negative.</li> </ul> If `e` is a datetime, `format` shall be + * a valid datetime pattern, see <a + * href="https://spark.apache.org/docs/latest/sql-ref-datetime-pattern.html">Datetime + * Patterns</a>. If `e` is a binary, it is converted to a string in one of the formats: <ul> + * <li>'base64': a base 64 string.</li> <li>'hex': a string in the hexadecimal format.</li> + * <li>'utf-8': the input binary is decoded to UTF-8 string.</li> </ul> ``` -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org