Skip first pass read of parquet data

[ray6080]

Current implementation unnecessarily read parquet data twice. The first pass is just to figure out num of rows per row group, for this, simply read the metadata is enough.

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (a0a51d5) 92.13% compared to head (9b50e11) 92.14%.

❗ Current head 9b50e11 differs from pull request most recent head 3a41372. Consider uploading reports for the commit 3a41372 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1480   +/-   ##
=======================================
  Coverage   92.13%   92.14%           
=======================================
  Files         670      670           
  Lines       23806    23806           
=======================================
+ Hits        21934    21936    +2     
+ Misses       1872     1870    -2     

Impacted Files	Coverage Δ
src/storage/copier/node_copier.cpp	93.52% <ø> (ø)
src/storage/copier/table_copy_executor.cpp	86.89% <100.00%> (ø)

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.