Skip to content

Conversation

wszhdshys
Copy link
Contributor

@wszhdshys wszhdshys commented Jul 16, 2025

What problem does this PR solve?

The octet function is implemented to calculate the number of bytes of the input value. And fixed the charlength function so that it correctly outputs the number of characters instead of bytes

What is changed and how it works?

Code changes

  • Has Rust code change
  • Has CI related scripts change

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Note for reviewer

…tes of the input value. And fixed the charlength function so that it correctly outputs the number of characters instead of bytes
@KKould KKould self-requested a review July 16, 2025 10:57
@KKould KKould added the enhancement New feature or request label Jul 16, 2025
@KKould KKould merged commit f7de8b9 into KipData:main Jul 16, 2025
3 checks passed
@KKould KKould mentioned this pull request Jul 16, 2025
48 tasks
@KKould KKould added this to the SQL 2016 milestone Jul 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants