mbutrovich commented on PR #17195:
URL: https://github.com/apache/datafusion/pull/17195#issuecomment-3192570690

   I'm not proud of the readability of the change, but at least there doesn't 
seem to be a meaningful performance regression:
   
   main
   ```
   regexp_replace_1000     time:   [1.5948 ms 1.5958 ms 1.5967 ms]
   
   regexp_replace_1000 utf8view
                           time:   [1.5956 ms 1.5966 ms 1.5976 ms]
   ```
   
   PR
   ```
   regexp_replace_1000     time:   [1.5982 ms 1.5991 ms 1.6000 ms]
   
   regexp_replace_1000 utf8view
                           time:   [1.6023 ms 1.6038 ms 1.6056 ms]
   ```


-- 
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: github-unsubscr...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to