manojkarthick commented on a change in pull request #9306:
URL: https://github.com/apache/arrow/pull/9306#discussion_r567465341
##########
File path: rust/parquet/README.md
##########
@@ -79,23 +79,23 @@ enabled by adding `RUSTFLAGS="-C target-feature=+sse4.2"`
before the
`cargo build` command.
## Test
-Run `cargo test` for unit tests.
+Run `cargo test` for unit tests. To also run tests related to the binaries,
use `cargo test --features cli`.
## Binaries
-The following binaries are provided (use `cargo install` to install them):
+The following binaries are provided (use `cargo install --features cli` to
install them):
- **parquet-schema** for printing Parquet file schema and metadata.
-`Usage: parquet-schema <file-path> [verbose]`, where `file-path` is the path
to a Parquet file,
-and optional `verbose` is the boolean flag that allows to print full metadata
or schema only
-(when not specified only schema will be printed).
+`Usage: parquet-schema <file-path>`, where `file-path` is the path to a
Parquet file. Use `-v/--verbose` flag
+to print full metadata or schema only (when not specified only schema will be
printed).
- **parquet-read** for reading records from a Parquet file.
`Usage: parquet-read <file-path> [num-records]`, where `file-path` is the path
to a Parquet file,
and `num-records` is the number of records to read from a file (when not
specified all records will
-be printed).
+be printed). Use `cargo install --features json_output` to enable `-j/--json`
flag for printing output
Review comment:
That was an outdated comment as I had removed that feature based on
earlier feedback. I've updated the PR to reflect your other comments as well.
Thanks @sunchao !
----------------------------------------------------------------
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]
