Aiee commented on PR #48048: URL: https://github.com/apache/arrow/pull/48048#issuecomment-3490352010
> Thanks for the PR, it feels too specific to me and prone to getting outdated soon if we end up doing any other spelling. I am not sure this is something we want to add to the documentation. @AlenkaF @thisisnic thoughts? In any case the linter is failing because those require double backticks ``""`` instead of single `""` See rendered docs not showing correctly here: http://crossbow.voltrondata.com/pr_docs/48048/cpp/csv.html#nulls Hi, thanks for the reply. My initial motivation is that I found it was hard to find the specific null spelling (also for `true_values` and `false_values') without diving into the source code. I thought it might be helpful to state it in the document since these values haven't been changed for a long time. But sure, it adds cost for future maintenance. -- 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]
