[string performence improvement]

[Veeupup]

I hereby agree to the terms of the CLA available at: https://databend.rs/policies/cla/

Summary

improve string function performance

• almost 50% speed improvement in release mode
• 100% extra memory copy improvement in base64/quote/trim/reverse string function

before:

now:

Changelog

• Performance Improvement

Related Issues

Fixes #3259

Test Plan

Unit Tests

Stateless Tests

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/databend/databend/FPa2TDzJkuyXDr1nnRJsAUtMQxpX
✅ Preview: Canceled

[Deployment for 4046f6b canceled]

[codecov-commenter]

Codecov Report

Merging #3288 (77d61f1) into main (7096d74) will increase coverage by 0%.
The diff coverage is 36%.

@@          Coverage Diff           @@
##            main   #3288    +/-   ##
======================================
  Coverage     65%     65%            
======================================
  Files        705     710     +5     
  Lines      38272   38634   +362     
======================================
+ Hits       24928   25183   +255     
- Misses     13344   13451   +107     

Impacted Files	Coverage Δ
common/datavalues/src/arrays/string/mod.rs	55% <ø> (ø)
common/functions/src/scalars/strings/base_64.rs	0% <0%> (ø)
common/functions/src/scalars/strings/quote.rs	0% <0%> (ø)
common/functions/src/scalars/strings/reverse.rs	0% <0%> (ø)
common/datavalues/src/arrays/string/transform.rs	44% <44%> (ø)
...mon/functions/src/scalars/strings/string2string.rs	60% <54%> (-9%)	⬇️
common/functions/src/scalars/strings/trim.rs	84% <69%> (-7%)	⬇️
common/datablocks/src/data_block.rs	75% <0%> (-14%)	⬇️
query/src/interpreters/interpreter_insert.rs	72% <0%> (-11%)	⬇️
common/datavalues/src/data_value.rs	39% <0%> (-3%)	⬇️
... and 28 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7096d74...77d61f1. Read the comment docs.

[sundy-li]

Is the performance in debug mode or release mode?

[Veeupup]

Is the performance in debug mode or release mode?

sorry, I test it in debug mode ... I try make it in release mode later

[databend-bot]

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

[sundy-li]

/LGTM

And I think it's time to add transfrom_from_primitive function.

[databend-bot]

Wait for another reviewer approval