thisisnic commented on issue #33405: URL: https://github.com/apache/arrow/issues/33405#issuecomment-4276118548
Reproducible in 23.0.1 and re-triaged with Claude - verify before fixing ---------------------------------------------------------------------------- The C++ CSV parser supports combining quoting + escaping, and Python passes both options correctly. However, the R binding in `csv___ParseOptions__initialize()` sets `escape_char` but never sets the escaping boolean — it's missing entirely. So escaping stays at its C++ default (false) regardless of what R users pass. Relevant code: https://github.com/apache/arrow/blob/b3362f2d241248a435ce53ae85bf3cf5c65d1432/r/src/csv.cpp#L60-L72 One-line fix: add res->escaping = cpp11::as_cpp<bool>(options["escaping"]); after the escape_char line. -- 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]
