[SPARK-46542] Remove the check for `c>=0` from `ExternalCatalogUtils#needsEscaping`

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.0.0
• Fix Version/s: 4.0.0
• Component/s: SQL
• Labels: pull-request-available
• Assignee: Yang Jie
• Reporter: Yang Jie
• Created: 29/Dec/23 07:32
• Updated: 30/Dec/23 05:14
• Resolved: 30/Dec/23 01:19

Description

 

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

 

 

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

Issue Links

• links to: GitHub Pull Request #44533