alamb commented on a change in pull request #289:
URL: https://github.com/apache/arrow-datafusion/pull/289#discussion_r628730469
##########
File path: datafusion-cli/src/main.rs
##########
@@ -51,6 +56,20 @@ pub async fn main() {
.validator(is_valid_batch_size)
.takes_value(true),
)
+ .arg(
+ Arg::with_name("file")
+ .help("Execute commands from file, then exit")
+ .short("f")
+ .long("file")
+ .validator(is_valid_file)
+ .takes_value(true),
+ )
+ .arg(
Review comment:
I can imagine other formats in the future other than CSV and pretty (for
example, JSON using the Arrow json writer)
What would you think about making the option `--format=csv` rather than
`--csv`, to leave room for future formats?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
