Skip to content

Cherry pick #103843 to 26.4: Preserve known identifiers in obfuscateQuery#103974

Merged
robot-ch-test-poll1 merged 1 commit intobackport/26.4/103843from
cherrypick/26.4/103843
May 6, 2026
Merged

Cherry pick #103843 to 26.4: Preserve known identifiers in obfuscateQuery#103974
robot-ch-test-poll1 merged 1 commit intobackport/26.4/103843from
cherrypick/26.4/103843

Conversation

@robot-ch-test-poll2
Copy link
Copy Markdown
Contributor

Original pull-request #103843

Do not merge this PR manually

This pull-request is a first step of an automated backporting.
It contains changes similar to calling git cherry-pick locally.
If you intend to continue backporting the changes, then resolve all conflicts if any.
Otherwise, if you do not want to backport them, then just close this pull-request.

The check results does not matter at this step - you can safely ignore them.

Troubleshooting

If the conflicts were resolved in a wrong way

If this cherry-pick PR is completely screwed by a wrong conflicts resolution, and you want to recreate it:

  • delete the pr-cherrypick label from the PR
  • delete this branch from the repository

You also need to check the Original pull-request for pr-backports-created label, and delete if it's presented there

The PR source

The PR is created in the CI job

@robot-ch-test-poll2 robot-ch-test-poll2 added pr-cherrypick Cherry-pick of merge-commit before backporting. Do not use manually - automated use only! do not test disable testing on pull request labels May 4, 2026
The original PR fixes both the SQL `obfuscateQuery` function (which does
not exist in 26.4) and `clickhouse-format --obfuscate`. The root-cause
fix in `TableFunctionFactory::isTableFunctionName` — consulting aliases
and the case-insensitive map in addition to the exact-case map — is the
only part that applies to 26.4 and is sufficient to preserve known table
function names (including aliases like `VALUES` and `url`) when running
`clickhouse-format --obfuscate`.

Original pull-request: #103843

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not test disable testing on pull request pr-cherrypick Cherry-pick of merge-commit before backporting. Do not use manually - automated use only!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants