Rafferty97 commented on PR #9468:
URL: https://github.com/apache/arrow-rs/pull/9468#issuecomment-3977012721

   > Thanks @Rafferty97 -- I haven't reviewed this PR very carefully, but it 
seems to me that the high level design is to transcode the csv data to utf8 and 
then use the existing parser
   > 
   > I think this would be possible to do today outside the CSV reader by using 
the decoder 
https://docs.rs/arrow-csv/latest/arrow_csv/reader/struct.Decoder.html
   > 
   > And converting the bytes to utf8 before feeding in
   > 
   > I would personally prefer adding an example to arrow-csv that shows how to 
do this rather than updating the API
   
   Thanks, I appreciate the feedback. I'll confirm whether that is the case and 
then close this PR if so, as that would indeed be preferable to changing the 
API.


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