Yang Jie created SPARK-46542:
--------------------------------

             Summary: Remove the check for `c>=0` from 
`ExternalCatalogUtils#needsEscaping`
                 Key: SPARK-46542
                 URL: https://issues.apache.org/jira/browse/SPARK-46542
             Project: Spark
          Issue Type: Improvement
          Components: SQL
    Affects Versions: 4.0.0
            Reporter: Yang Jie


 
{code:java}
def needsEscaping(c: Char): Boolean = {
  c >= 0 && c < charToEscape.size() && charToEscape.get(c)
} {code}
 

 

The numerical range of Char in Scala is from 0 to 65,535, so `c>=0` is always 
true.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to