[FLINK-32466] Invalid input strategy for many functions which allows BINARY strings

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.18.0
• Component/s: Table SQL / Planner
• Labels: pull-request-available
• Assignee: Timo Walther
• Reporter: Timo Walther
• Created: 28/Jun/23 09:35
• Updated: 10/Jul/23 17:12
• Resolved: 10/Jul/23 17:12

Description

"string" in SQL terms covers both character strings and binary strings. The author of CONCAT might not have known this. In any case, the code gen instead of the validator fails when executing:

TableEnvironment t = TableEnvironment.create(EnvironmentSettings.inStreamingMode());
t.createTemporaryView("t", t.fromValues(lit(new byte[] {97})));
t.executeSql("SELECT CONCAT(f0, '-magic') FROM t").print();

As future work, we should also allow binary strings.

Issue Links

• links to: GitHub Pull Request #22892