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

   ## Which issue does this PR close?
   
   Closes #11393 .
   
   ## Rationale for this change
   
   This is a much clearer error message than previously existed in its place. 
   
   ## What changes are included in this PR?
   
   This changes the error message and adds two tests: one to cover this exact 
use case (of no statements being provided) and another to cover the instance of 
multiple statements being provided (which returns an error right before the 
check to see if any statements were provided at all), since there were no tests 
for that error case.
   
   ## Are these changes tested?
   
   Yes, running `cargo test` in the root of the repo produces no failures.
   
   ## Are there any user-facing changes?
   
   I think this qualifies as a user-facing change, but I don't think it 
violates any invariants or changes any use-cases (or anything like that) in a 
way that would require any documentation changes. I may be wrong, though - 
clarification on if this qualifies as a 'user-facing change' would be 
appreciated.
   


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to