[fix](doc) v4.x: remove stray control characters that break HTML minifier#3843
Merged
dataroaring merged 1 commit intoMay 29, 2026
Merged
Conversation
…fier STRUCT.md had a trailing 0x03 after `DISTRIBUTED BY HASH(id) BUCKETS 1`; sequence-count.md and sequence-match.md had 0x19 in place of the apostrophe in "isn't". The HTML minifier rejects control characters in the input stream, failing the build for these pages. Fixed in both docs/ (next) and versioned_docs/version-4.x/.
Contributor
There was a problem hiding this comment.
Pull request overview
Removes stray control characters from a small set of SQL manual Markdown pages that were causing Docusaurus’ HTML minifier to fail during site builds (notably affecting the 4.x versioned docs and “next” docs).
Changes:
- Replaced the invalid control character used in “isn't” with a normal ASCII apostrophe in
sequence-countandsequence-matchdocs. - Removed a trailing
0x03control byte from a SQL example line in theSTRUCTdocs. - Applied the same cleanup in both
docs/(next) andversioned_docs/version-4.x/(4.x).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| versioned_docs/version-4.x/sql-manual/sql-functions/aggregate-functions/sequence-match.md | Replaces control-character apostrophes to prevent minifier failures in 4.x docs. |
| versioned_docs/version-4.x/sql-manual/sql-functions/aggregate-functions/sequence-count.md | Replaces control-character apostrophes to prevent minifier failures in 4.x docs. |
| versioned_docs/version-4.x/sql-manual/basic-element/sql-data-types/semi-structured/STRUCT.md | Removes trailing control byte from a SQL snippet line in 4.x docs. |
| docs/sql-manual/sql-functions/aggregate-functions/sequence-match.md | Replaces control-character apostrophes to prevent minifier failures in “next” docs. |
| docs/sql-manual/sql-functions/aggregate-functions/sequence-count.md | Replaces control-character apostrophes to prevent minifier failures in “next” docs. |
| docs/sql-manual/basic-element/sql-data-types/semi-structured/STRUCT.md | Removes trailing control byte from a SQL snippet line in “next” docs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
STRUCT.md(semi-structured data types) had a trailing0x03byte afterDISTRIBUTED BY HASH(id) BUCKETS 1on line 147.sequence-count.mdandsequence-match.md(aggregate functions) had0x19standing in for the apostrophe in "isn't" (three occurrences across the two files)./cloud/26.x/...and/enterprise/4.x/...URL paths.docs/(next) andversioned_docs/version-4.x/. Chinese (i18n) copies and other version trees were already clean.Test plan
yarn build(or the project's site build) completes without the previousControl character in input streamerrors on the three affected pages.enterprise/4.xandcloud/26.xand confirm "isn't" displays with a normal apostrophe and theDISTRIBUTED BY ... BUCKETS 1line renders cleanly.🤖 Generated with Claude Code