GH-3289: Introduce Parquet E2E test harness to support CLI testing#3296
GH-3289: Introduce Parquet E2E test harness to support CLI testing#3296ArnavBalyan wants to merge 6 commits into
Conversation
|
cc @gszadovszky as followup thanks! |
5d453a4 to
497858c
Compare
gszadovszky
left a comment
There was a problem hiding this comment.
Thank you @ArnavBalyan for picking this up. It is a great improvement to improve the test coverage of the cli!
|
cc @gszadovszky this is ready thanks! |
|
@ArnavBalyan, could you check the failures? |
gszadovszky
left a comment
There was a problem hiding this comment.
Thanks, @ArnavBalyan.
|
Oh, the tests are still failing... |
|
Strangely I am not seeing it locally let me fix it |
3f7caa8 to
57011ae
Compare
|
@ArnavBalyan Gentle ping! Did you find some time to fix the test? |
|
Hi @Fokko thanks for the ping, I'll revive it in the coming week, looks like it was a formatting issue. Will try to fix soon! |
|
This pull request has been automatically marked as stale because it has had no activity for at least 2 months. If you are still working on this change or plan to move it forward, please leave a comment or push a new commit so we know to keep it open. Otherwise, this PR will be closed automatically in about one month. Thank you for your contribution to Apache Parquet! |
linecount,partial matchetc.