[ 
https://issues.apache.org/jira/browse/SPARK-23892?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16429253#comment-16429253
 ] 

Apache Spark commented on SPARK-23892:
--------------------------------------

User 'kiszk' has created a pull request for this issue:
https://github.com/apache/spark/pull/21000

> Improve coverage and fix lint error in UTF8String-related Suite
> ---------------------------------------------------------------
>
>                 Key: SPARK-23892
>                 URL: https://issues.apache.org/jira/browse/SPARK-23892
>             Project: Spark
>          Issue Type: Improvement
>          Components: Spark Core
>    Affects Versions: 2.3.0
>            Reporter: Kazuaki Ishizaki
>            Priority: Minor
>
> The following code in {{UTF8StringSuite}} has no sense.
> {code}
>     assertTrue(s1.startsWith(s1));
>     assertTrue(s1.endsWith(s1));
> {code}
> The code {{if (length <= 0) ""}} in {{UTF8StringPropertyCheckSuite}} has no 
> sense
> {code}
>   test("lpad, rpad") {
>     def padding(origin: String, pad: String, length: Int, isLPad: Boolean): 
> String = {
>       if (length <= 0) return ""
>       if (length <= origin.length) {
>         if (length <= 0) "" else origin.substring(0, length)
>       } else {
>        ...
> {code}
> The previous change in {{UTF8StringSuite}} broke lint-java check.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to