[ https://issues.apache.org/jira/browse/SPARK-46542?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
ASF GitHub Bot updated SPARK-46542: ----------------------------------- Labels: pull-request-available (was: ) > 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 > Priority: Minor > Labels: pull-request-available > > > {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