Reranko05 commented on code in PR #51038: URL: https://github.com/apache/arrow/pull/51038#discussion_r4071907197
########## cpp/src/arrow/util/simdjson_internal.cc: ########## @@ -581,4 +581,23 @@ Status ValidateJsonDocument(simdjson::ondemand::parser& parser, return ConsumeJsonValue(value); } +// XXX We could try to SIMD-accelerate this routine but it's called only +// once per chunk and also will presumably examine a minimal amount of bytes. Review Comment: Agreed. Updated the comment and added a comment explaining the what it does. -- 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]
