Re: [PR] [SPARK-59379][CORE] Add an ASCII fast-path to UTF8String position lookups, warmed by numChars() [spark]

["david-mollitor-db (via GitHub)" <gi...@apache.org>]

david-mollitor-db commented on PR #58668:
URL: https://github.com/apache/spark/pull/58668#issuecomment-5776991850

   Thanks @uros-b — combined both changes here and dropping #58667. This PR now contains the `numChars()` flag caching (from #58667) plus the ASCII fast-path that reads it, squashed into a single commit so the pair travels together. Rebased on master and re-pushed; `UTF8StringSuite` (53 tests) passes. PTAL when you have a moment.
   


-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org