fix missing spaces when formatting unlock snapshot

[hanfei1991]

Changelog category (leave one):

• Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

fix missing spaces when formatting unlock snapshot. close #101723

Documentation entry for user-facing changes

• Documentation is written (mandatory for new features)

[clickhouse-gh]

Workflow [PR], commit [b7d34d5]

Summary: ❌

job_name	test_name	status	info	comment
Unit tests (tsan)		failure
	FunctionsStress.stress	failure	cidb
	AllTests	failure	cidb

---

AI Review

Summary

This PR fixes SQL formatter output for UNLOCK SNAPSHOT by adding missing spaces in two formatter paths (ALTER ... UNLOCK SNAPSHOT and SYSTEM UNLOCK SNAPSHOT) and adds a stateless test covering both statements. I did not find correctness, safety, performance, or compatibility issues in the patch.

ClickHouse Rules

Item	Status	Notes
Deletion logging	➖
Serialization versioning	➖
Core-area scrutiny	✅
No test removal	✅
Experimental gate	➖
No magic constants	✅
Backward compatibility	✅
SettingsChangesHistory.cpp	➖
PR metadata quality	✅
Safe rollout	✅
Compilation time	✅
No large/binary files	✅

Final Verdict

• Status: ✅ Approve

[alexey-milovidov]

The flaky check failure is fixed in #102148, let's update the branch.

[clickhouse-gh]

LLVM Coverage Report

Metric	Baseline	Current	Δ
Lines	84.00%	84.00%	+0.00%
Functions	90.80%	90.90%	+0.10%
Branches	76.50%	76.50%	+0.00%

Changed lines: 92.86% (13/14) | lost baseline coverage: 2 line(s) · Uncovered code

Full report · Diff report