SEZ9 commented on issue #12669:
URL: https://github.com/apache/seatunnel/issues/12669#issuecomment-6052554747

   Thanks for the clear report and reproduction. The root cause as described is 
plausible: the serializer joins fields with the configured delimiter, so using 
a comma-default CSVPrinter to decide MINIMAL quoting is inconsistent, and a 
value like `a|b` ends up unquoted and shifts the columns on read.
   
   Configuring the printer with the same delimiter character the reader splits 
on sounds like the right minimal fix. The round-trip coverage in 
CsvTextFormatSchemaTest and the LocalFile write/read/assert scenario are both 
valuable here, since a row-count-only check would miss the column shift. Let's 
keep this issue open until the linked PR passes CI and review.
   
   <!-- streview-comment:1603 -->


-- 
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]

Reply via email to