taepper commented on code in PR #51758:
URL: https://github.com/apache/arrow/pull/51758#discussion_r4196070289


##########
cpp/src/arrow/json/parser.cc:
##########


Review Comment:
   We could solve this like in this case:
   https://github.com/apache/arrow/pull/51472
   
   where we let `simdjson` handle the skip by the regular advancing of the 
iterator
   
   But this would avoid the json validation (beyond structural errors regarding 
braces `[`,`]`,`{`,`}` which will still be checked), which is probably required 
at this point



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