[SPARK-41173] Move `require()` out from the constructors of string expressions

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.4.0
• Component/s: SQL
• Assignee: Yang Jie
• Reporter: Max Gekk
• Created: 17/Nov/22 13:38
• Updated: 18/Nov/22 19:15
• Resolved: 18/Nov/22 19:15

Description

1. ConcatWs:
https://github.com/apache/spark/blob/fabea7101ea55db991590ca2fbe1d4dfd25e5b28/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala#L70
2. FormatString
https://github.com/apache/spark/blob/fabea7101ea55db991590ca2fbe1d4dfd25e5b28/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala#L1665

Issue Links

• links to: [Github] Pull Request #38705 (LuciferYang)