minor: make Parquet CLI input args consistent

parquet/pytest/test_parquet_integration.py

@@ -68,15 +68,13 @@ def get_show_filter_cli_output(output_dir, data, col_name="id"):
     (parquet_file,) = sorted(pathlib.Path(output_dir).glob("*.parquet"))
     args = [
         "parquet-show-bloom-filter",
-        "--file-name",
         parquet_file,
-        "--column",
         col_name,
     ]
     for v in data:
-        args.extend(["--values", v[0]])
+        args.extend([v[0]])
     for v in data:
-        args.extend(["--values", v[1]])
+        args.extend([v[1]])
     return subprocess.check_output(args)
 
 

parquet/src/bin/parquet-read.rs

@@ -45,7 +45,7 @@ use std::{fs::File, path::Path};
 #[derive(Debug, Parser)]
 #[clap(author, version, about("Binary file to read data from a Parquet file"), long_about = None)]
 struct Args {
-    #[clap(short, long, help("Path to a parquet file, or - for stdin"))]
+    #[clap(help("Path to a parquet file, or - for stdin"))]
     file_name: String,
     #[clap(
         short,

parquet/src/bin/parquet-rowcount.rs

@@ -44,8 +44,6 @@ use std::{fs::File, path::Path};
 #[clap(author, version, about("Binary file to return the number of rows found from Parquet file(s)"), long_about = None)]
 struct Args {
     #[clap(
-        short,
-        long,
         number_of_values(1),
         help("List of Parquet files to read from separated by space")
     )]

parquet/src/bin/parquet-schema.rs

@@ -46,7 +46,7 @@ use std::{fs::File, path::Path};
 #[derive(Debug, Parser)]
 #[clap(author, version, about("Binary file to print the schema and metadata of a Parquet file"), long_about = None)]
 struct Args {
-    #[clap(short, long)]
+    #[clap(help("Path to the parquet file"))]
     file_path: String,
     #[clap(short, long, help("Enable printing full file metadata"))]
     verbose: bool,

parquet/src/bin/parquet-show-bloom-filter.rs

@@ -25,7 +25,7 @@
 //! ```
 //! After this `parquet-show-bloom-filter` should be available:
 //! ```
-//! parquet-show-bloom-filter --file-name XYZ.parquet --column id --values a
+//! parquet-show-bloom-filter XYZ.parquet id a
 //! ```
 //!
 //! The binary can also be built from the source code and run as follows:
@@ -44,17 +44,11 @@ use std::{fs::File, path::Path};
 #[derive(Debug, Parser)]
 #[clap(author, version, about("Binary file to read bloom filter data from a Parquet file"), long_about = None)]
 struct Args {
-    #[clap(short, long, help("Path to the parquet file"))]
+    #[clap(help("Path to the parquet file"))]
     file_name: String,
-    #[clap(
-        short,
-        long,
-        help("Check the bloom filter indexes for the given column")
-    )]
+    #[clap(help("Check the bloom filter indexes for the given column"))]
     column: String,
     #[clap(
-        short,
-        long,
         help("Check if the given values match bloom filter, the values will be evaluated as strings"),
         required = true
     )]