[
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
