fix(python): Pass skip_rows_after_header to pyarrow csv reader

[reswqa]

This fixes #15497.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.13%. Comparing base (cc6c642) to head (23e10b4).
Report is 13 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #15533      +/-   ##
==========================================
- Coverage   81.14%   81.13%   -0.02%     
==========================================
  Files        1362     1363       +1     
  Lines      174989   175161     +172     
  Branches     2533     2527       -6     
==========================================
+ Hits       142000   142110     +110     
- Misses      32505    32573      +68     
+ Partials      484      478       -6     

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

[alexander-beedie]

Ah; I think the unit test needs to check both cases? (eg: use_pyarrow = True/False; the issue mentions True, but the test checks False). The fix itself looks good though 👍

[reswqa]

Oops, I want to check the pyarrow = True indeed, good catch! But it's better to test both, updated 😉

[alexander-beedie]

Oops, I want to check the pyarrow = True indeed, good catch! But it's better to test both, updated 😉

Perfect; will merge when CI goes green :)