saleemno1 commented on PR #626:
URL: https://github.com/apache/commons-csv/pull/626#issuecomment-5262804812
Fair to want that nailed down. The reason I read it as a bug and not just
undefined: it is a round trip through the library's own printer and parser with
the same predefined format. For MySQL/PostgreSQLText/Oracle, `printRecord("\N",
null)` writes the value as `\\N` and the null as `\N`, so the printer already
escapes the value specifically to keep it distinct from the null marker. On the
way back the parser dropped that distinction and both fields came back as null,
so a real value silently became null.
So the two halves of the library disagree on the same format, and the escape
the printer itself emitted is exactly what tells them apart. That is all the
patch keys off. `testEscapedNullStringIsAValue` in the PR is that round trip
and fails on master today.
Totally your call on scope, but that is why it looks like a correctness gap
rather than open behavior to me.
--
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]
