`translateUTF8`: Fix slightly incorrect error message by rschu1ze · Pull Request #104817 · ClickHouse/ClickHouse · GitHub
Skip to content

translateUTF8: Fix slightly incorrect error message#104817

Merged
leshikus merged 3 commits into
masterfrom
fix-104786
May 20, 2026
Merged

translateUTF8: Fix slightly incorrect error message#104817
leshikus merged 3 commits into
masterfrom
fix-104786

Conversation

@rschu1ze

@rschu1ze rschu1ze commented May 13, 2026 •
edited by robot-clickhouse
Loading

Copy link
Copy Markdown
Member

The error message now complains about the code point count instead of the byte count.

Resolves: #104786

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Version info

  • Merged into: 26.5.1.918

@rschu1ze
translateUTF8: Fix slightly incorrect error message
6eaac45 
The error message now complains about the code point count instead of
the byte count.
@clickhouse-gh

clickhouse-gh Bot commented May 13, 2026 •
edited by rschu1ze
Loading

Copy link
Copy Markdown
Contributor

@clickhouse-gh clickhouse-gh Bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 13, 2026
@Algunenano Algunenano self-assigned this May 13, 2026
@Algunenano

Algunenano commented May 13, 2026

Copy link
Copy Markdown
Member

Fuzzer issue fixed here: #104858

@alexey-milovidov alexey-milovidov mentioned this pull request May 17, 2026
Merged
1 task
@alexey-milovidov

alexey-milovidov commented May 17, 2026

Copy link
Copy Markdown
Member

This was fixed by #105146. Let's update the branch.

@clickhouse-gh

clickhouse-gh Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor

LLVM Coverage Report

Metric Baseline Current Δ
Lines 84.10% 84.10% +0.00%
Functions 91.40% 91.40% +0.00%
Branches 76.50% 76.60% +0.10%

Changed lines: 100.00% (4/4) · Uncovered code

Full report · Diff report

@rschu1ze

rschu1ze commented May 20, 2026

Copy link
Copy Markdown
Member Author

@leshikus leshikus added this pull request to the merge queue May 20, 2026
Merged via the queue into master with commit 0797c0c May 20, 2026
162 of 165 checks passed
@leshikus leshikus deleted the fix-104786 branch May 20, 2026 22:11
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label May 20, 2026
DavidHe-2008 pushed a commit to DavidHe-2008/ClickHouse that referenced this pull request Jun 1, 2026
@leshikus
Merge pull request ClickHouse#104817 from ClickHouse/fix-104786
62d5c0b 
`translateUTF8`: Fix slightly incorrect error message
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 mentioned this pull request Jun 27, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Algunenano Algunenano Algunenano approved these changes

Assignees

@Algunenano Algunenano

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

translateUTF8 error message reports byte sizes instead of code point counts

5 participants

@rschu1ze @Algunenano @alexey-milovidov @leshikus @robot-ch-test-poll