Inf1mum commented on issue #27239:
URL: https://github.com/apache/datafusion/issues/27239#issuecomment-6834566492

   Unclaiming `encoding.slt` and `expr.slt`. All 4 matching lines (encoding.slt 
65, 68, 83 and expr.slt 608) fail step 3 of the recipe: the actual error is 
longer than the old regex. For example, `select encode(12, 'hex');` now fails 
with
   
   ```
   DataFusion error: Error during planning: Function 'encode' requires Binary, 
but received Int64 (DataType: Int64).. No function matches the given name and 
argument types 'encode(Int64, Utf8)'. You might need to add explicit type casts.
        Candidate functions:
        encode(Binary, String)
   ```
   
   and `decode` (encoding.slt 68), `to_hex` (encoding.slt 83), and `repeat` 
(expr.slt 608) have the same appended signature text. The inline regexes match 
only a prefix of these messages, so the plain-text record cannot equal the old 
regex with the escapes removed. The appended "No function matches …" text and 
these exact slt lines both go back to 
[#20605](https://github.com/apache/datafusion/pull/20605), so the regexes have 
been prefix-matches ever since. Converting them would mean rewriting the checks 
to the full multi-line message rather than a mechanical rewrite, so I'm 
releasing the claim instead of opening an empty PR.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to