itang06 opened a new pull request, #27264:
URL: https://github.com/apache/datafusion/pull/27264

   ## Which issue does this PR close?
   
   Part of #27239. This is the [five-file batch claimed in the 
issue](https://github.com/apache/datafusion/issues/27239#issuecomment-6741109824).
   
   ## Rationale for this change
   
   Plain-text error blocks make full error messages easier to read and review.
   
   ## What changes are included in this PR?
   
   Convert 28 escaped-regex checks across `scalar.slt` (1), `explain.slt` (4), 
`order.slt` (3), `select.slt` (16), and `joins.slt` (4). Keep the `strpos` 
regex in `scalar.slt` because the actual error also includes casting advice and 
candidate signatures.
   
   ## What is the testing strategy for this PR?
   
   The five files passed before and after the conversion. I used `--complete` 
on each file and kept a block only when its generated message matched the 
unescaped original. The final run used normal test mode.
   
   ```sh
   cargo test --test sqllogictests -- test_files/scalar.slt 
test_files/explain.slt test_files/order.slt test_files/select.slt 
test_files/joins.slt
   ```
   
   ## Are there any user-facing changes?
   
   This PR changes test expectations only.
   
   This batch was prepared with AI assistance and verified with the DataFusion 
test runner and local lint suite.
   
   @jayzhan211 This is my first DataFusion PR. Could you approve the CI run if 
GitHub requires it?
   


-- 
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