reader: remove counting of blocks

[Riolku]

This allows us to move toward removing all upfront counting. None of the readers actually depend on upfront counting of the number of blocks.

Depends on #2156 , to avoid a merge conflict.

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (abca5e6) 89.92% compared to head (67cbaca) 90.04%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2166      +/-   ##
==========================================
+ Coverage   89.92%   90.04%   +0.11%     
==========================================
  Files         989      989              
  Lines       35637    35452     -185     
==========================================
- Hits        32048    31921     -127     
+ Misses       3589     3531      -58     

Files	Coverage Δ
src/common/arrow/arrow_converter.cpp	97.24% <ø> (+27.24%)	⬆️
src/common/types/date_t.cpp	89.62% <ø> (ø)
src/common/types/dtime_t.cpp	74.68% <ø> (ø)
src/common/types/types.cpp	92.25% <100.00%> (-0.02%)	⬇️
src/function/base_lower_upper_operation.cpp	100.00% <100.00%> (ø)
src/include/common/types/cast_helpers.h	87.14% <ø> (ø)
...include/function/arithmetic/arithmetic_functions.h	93.22% <100.00%> (+0.11%)	⬆️
src/include/function/boolean/boolean_functions.h	85.71% <100.00%> (-3.18%)	⬇️
...r/operator/persistent/reader/csv/base_csv_reader.h	100.00% <ø> (ø)
...e/processor/operator/persistent/reader_functions.h	100.00% <100.00%> (ø)
... and 20 more

... and 3 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ray6080]

Can we also add isEOF() to serial csv reader too? I wonder if that can simplify the control logic by eventually remove doneAfterEmptyBlock later.

[Riolku]

By removing doneAfterEmptyBlock, you mean change this to a universal isEOF method?