JingsongLi commented on PR #9904: URL: https://github.com/apache/paimon/pull/9904#issuecomment-5708351229
The premise checks out: `CsvFormatWriter` appends the delimiter/quote/escape option strings verbatim, while `CsvParser` only ever reads `charAt(0)` — a multi-character value silently corrupted the read path, and there was no validation at all. Validating at option construction is the right place, and keeping `LINE_DELIMITER` exempt is correct since it is genuinely a multi-character token. The test covers both a multi-char value and the empty string, and the docs are updated consistently. -- 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]
