Skip to content

[fix](typo) fix spelling typos in BE and cloud comments and messages#65371

Open
spaces-X wants to merge 1 commit into
apache:masterfrom
spaces-X:fix-cpp-comment-string-typos
Open

[fix](typo) fix spelling typos in BE and cloud comments and messages#65371
spaces-X wants to merge 1 commit into
apache:masterfrom
spaces-X:fix-cpp-comment-string-typos

Conversation

@spaces-X

@spaces-X spaces-X commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

Fix common spelling mistakes in code comments and a few user-facing log/error messages under be/src and cloud/src.

Examples: funcitonfunction, seperateseparate, unkownunknown, invaildinvalid, avaliableavailable, sucesssuccess, retrunreturn, aquireacquire, shoudshould, wetherwhether, defalutdefault, proccessprocess.

It also renames one locally misspelled variable colunmcolumn in segment_iterator.cpp (3 occurrences in a single loop scope, no conflict).

Scope / safety

  • 34 files, 46 lines, all under be/src and cloud/src.
  • Comments and string literals only (plus the one local variable rename); no functional change.
  • The whole-word replacement is underscore-aware, so it does not touch compound identifiers (e.g. test names like invaild_jsonb_document, test_signal_aquire) or intentional test data (e.g. unkown_resource_xxxxxx).
  • Verified that none of the corrected message strings are asserted by any unit or regression test.

Types of changes

  • Bugfix / typo (non-breaking change which fixes an issue)

@hello-stephen

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

Fix common spelling mistakes (e.g. funciton -> function, seperate ->
separate, unkown -> unknown, invaild -> invalid, avaliable -> available,
sucess -> success) in code comments and in a few user-facing log/error
messages under be/src and cloud/src. Also rename a locally misspelled
variable `colunm` -> `column` in segment_iterator.cpp.

Comments and string literals only; no functional change. Verified that
none of the corrected message strings are asserted by unit or regression
tests.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@spaces-X spaces-X force-pushed the fix-cpp-comment-string-typos branch from 46ee32a to 801b0c9 Compare July 8, 2026 09:23
@github-actions github-actions Bot added the approved Indicates a PR has been approved by one committer. label Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants