JeremyXin commented on PR #12768: URL: https://github.com/apache/seatunnel/pull/12768#issuecomment-6218990507
@DanielLeens @DanielLeens Thanks for the follow-up. I pushed 7db7df042 with Option B. Excel header mapping now requires excel_use_header_line = true (default false). Setting skip_header_row_number alone preserves positional reading, including for partially matching headers. When enabled, both POI and EasyExcel use the first row for name-based mapping; missing schema fields produce null with a warning. I updated the LocalFile docs in English and Chinese, the unit tests for both engines, and the Excel E2E configurations. The tests cover the legacy partial-match case and the new option’s precedence over skip_header_row_number. Please help me review it again. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
