Aiee commented on PR #48048:
URL: https://github.com/apache/arrow/pull/48048#issuecomment-3495649300

   > It is a bit of a paint to find relevant information for this specific case.
   > 
   > What if we update the header file here:
   > 
   > 
https://github.com/apache/arrow/blob/5eaf553bfc7aa639fd67bd622b6b808e71fbba39/cpp/src/arrow/csv/options.h#L130-L132
   > 
   > with additional information about what are "default null values" (plus 
others) and then link the API docs section here: 
[arrow.apache.org/docs/cpp/api/formats.html#_CPPv4N5arrow3csv14ConvertOptions8DefaultsEv](https://arrow.apache.org/docs/cpp/api/formats.html#_CPPv4N5arrow3csv14ConvertOptions8DefaultsEv)
   > 
   > to the Nulls section in the User Guide? 
http://crossbow.voltrondata.com/pr_docs/48048/cpp/csv.html#nulls
   > 
   > Adding such a specific list directly to the docs does look like a possible 
future issue with outdated information.
   
   Hi, I've reverted the doc change and update the header file as suggested. 
Please take a look.


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