zhangxinyao88 opened a new pull request, #3858:
URL: https://github.com/apache/parquet-java/pull/3858

   ### Rationale for this change
   
   Continue #3289 with CLI coverage for `head`, `cat`, and `scan`. Existing 
command tests mostly set fields directly and check return codes, leaving 
argument parsing and observable results untested.
   
   ### What changes are included in this PR?
   
   Add six tests using the existing CLI helper and deterministic Parquet files:
   - `head` defaults to ten records and honors `--num-records`.
   - `cat` prints all records and honors `--columns`.
   - `scan` counts records across multiple input files and rejects a missing 
column.
   
   Record output is parsed and compared by field and value. Scan assertions 
check counts and the missing-field diagnostic without depending on elapsed time 
or complete log output.
   
   ### Are these changes tested?
   
   `./mvnw -B -ntp -pl parquet-cli spotless:check test`: 87 passed, 2 skipped, 
including all six new tests. Spotless and license checks passed.
   
   ### Are there any user-facing changes?
   
   No. Tests only.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to