viirya commented on PR #5235:
URL: https://github.com/apache/arrow-rs/pull/5235#issuecomment-1868071898

   > As an aside it occurs to me we should probably have a version of this that 
takes `Datum`
   
   Yea, I did a local version and tested it with DataFusion. It turns out the 
performance bottleneck is on cloning Regex mostly. But it is per query 
benchmark and I didn't run this kernel benchmark in it.
   
   I will run this benchmark against the Datum version and probably submit it 
later.


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