Refactor ParquetNativeRecordReader to have all the types to support

[xiangfu0]

• Restructure the types to implement for Parquet data type converter
• Adding more test cases for parquet files

[codecov-commenter]

Codecov Report

Merging #9352 (b5cf587) into master (0f4bcfc) will decrease coverage by 43.66%.
The diff coverage is 0.00%.

❗ Current head b5cf587 differs from pull request most recent head 1d3e335. Consider uploading reports for the commit 1d3e335 to get more accurate results

@@              Coverage Diff              @@
##             master    #9352       +/-   ##
=============================================
- Coverage     69.80%   26.13%   -43.67%     
+ Complexity     4777       44     -4733     
=============================================
  Files          1875     1872        -3     
  Lines         99860    99914       +54     
  Branches      15194    15212       +18     
=============================================
- Hits          69706    26115    -43591     
- Misses        25231    71180    +45949     
+ Partials       4923     2619     -2304     

Flag	Coverage Δ
integration1	26.13% <0.00%> (-0.07%)	⬇️
integration2	?
unittests1	?
unittests2	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...utformat/parquet/ParquetNativeRecordExtractor.java	0.00% <0.00%> (-68.00%)	⬇️
...in/java/org/apache/pinot/spi/utils/BytesUtils.java	0.00% <0.00%> (-100.00%)	⬇️
.../java/org/apache/pinot/spi/utils/BooleanUtils.java	0.00% <0.00%> (-100.00%)	⬇️
...java/org/apache/pinot/spi/trace/BaseRecording.java	0.00% <0.00%> (-100.00%)	⬇️
...java/org/apache/pinot/spi/trace/NoOpRecording.java	0.00% <0.00%> (-100.00%)	⬇️
...ava/org/apache/pinot/spi/config/table/FSTType.java	0.00% <0.00%> (-100.00%)	⬇️
...ava/org/apache/pinot/spi/config/user/RoleType.java	0.00% <0.00%> (-100.00%)	⬇️
...ava/org/apache/pinot/spi/data/MetricFieldSpec.java	0.00% <0.00%> (-100.00%)	⬇️
...ava/org/apache/pinot/spi/utils/NullValueUtils.java	0.00% <0.00%> (-100.00%)	⬇️
...java/org/apache/pinot/common/tier/TierFactory.java	0.00% <0.00%> (-100.00%)	⬇️
... and 1376 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[Jackie-Jiang]

Do we need to check in so many test files? Having a test file with all the field type should be enough?

[xiangfu0]

Do we need to check in so many test files? Having a test file with all the field type should be enough?

Those are various small parquet files from:
https://github.com/apache/parquet-testing/tree/master/data
https://github.com/dask/fastparquet/tree/main/test-data

I copied over them and wanna make sure we pass all of them until we found a better test plan.

We still have files don't support yet e.g. compression lib missing or encryption methods etc.
I commented out those files in the tests, the goal is to uncomment them all.