add param validation for datafusion-cli

[jimexist]

Which issue does this PR close?

Closes #283.

Rationale for this change

More specific error messages help users understand how to fix the problem

Are there any user-facing changes?

better error messages

[jimexist]

❯ cargo run --release --bin datafusion-cli -- -p 123
    Finished release [optimized] target(s) in 0.24s
     Running `target/release/datafusion-cli -p 123`
error: Invalid value for '--data-path <data-path>': Invalid data directory “123”

[jimexist]

❯ cargo run --release --bin datafusion-cli -- -c 0
    Finished release [optimized] target(s) in 0.20s
     Running `target/release/datafusion-cli -c 0`
error: Invalid value for '--batch-size <batch-size>': Invalid batch size “0”

[jimexist]

need to fix #286 first

[alamb]

Thanks @jimexist -- this looks great!

[codecov-commenter]

Codecov Report

Merging #284 (8ed8fe7) into master (b8805d4) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #284      +/-   ##
==========================================
- Coverage   76.19%   76.16%   -0.03%     
==========================================
  Files         140      140              
  Lines       23595    23603       +8     
==========================================
  Hits        17978    17978              
- Misses       5617     5625       +8     

Impacted Files	Coverage Δ
datafusion-cli/src/main.rs	0.00% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b8805d4...8ed8fe7. Read the comment docs.

[jorgecarleitao]

Thank you, @jimexist . This looks good. 👍

[Dandandan]

Thanks @jimexist !!