DanielLeens commented on PR #10257: URL: https://github.com/apache/seatunnel/pull/10257#issuecomment-5011961333
Thanks for the original contribution. I picked this up and rebased the branch onto the latest `dev`, then completed the fix so the CSV header uses both the configured field delimiter and encoding consistently with data rows. I also replaced the write-only E2E coverage with direct output assertions for the default comma, a multi-character delimiter, and UTF-16LE with non-ASCII header/data. The effective PR diff is now limited to `CsvWriteStrategy` and its regression test. GitHub CI has been triggered on the updated head. Thank you again for identifying this issue and contributing the initial fix. -- 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]
