Re: [PR] datafusion-cli support for multiple commands in a single line [arrow-datafusion]

["comphead (via GitHub)" <gi...@apache.org>]

comphead commented on code in PR #9831:
URL: https://github.com/apache/arrow-datafusion/pull/9831#discussion_r1543908224


##########
datafusion-cli/src/helper.rs:
##########
@@ -197,6 +204,19 @@ pub fn unescape_input(input: &str) -> datafusion::error::Result<String> {
     Ok(result)
 }
 
+/// Splits a string which consists of multiple queries.
+pub(crate) fn split_from_semicolon(sql: String) -> Vec<String> {
+    sql.split(';')

Review Comment:
   We may want to add a test? Just wondering if it will work for query 
   ```
   select ';'
   ```
   ?



-- 
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-unsubscribe@arrow.apache.org

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