Digest text changes SQL semantic

[breezewish]

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

Execute SQL statement:

SELECT SUM(TABLE_ROWS) FROM INFORMATION_SCHEMA.TABLES;

2. What did you expect to see? (Required)

Expect a statement digest text that does not have syntax errors, like

SELECT SUM(TABLE_ROWS) FROM INFORMATION_SCHEMA.TABLES

in INFORMATION_SCHEMA.CLUSTER_STATEMENTS_SUMMARY_HISTORY table.

3. What did you see instead (Required)

select sum ( table_rows ) from information_schema . tables

Note that this SQL statement will throw syntax error:

1064 - You have an error in your SQL syntax; check the manual that corresponds to your TiDB version for the right syntax to use line 1 column 12 near "( table_rows ) from information_schema . tables" , Time: 0.000000s

4. Affected version (Required)

My version is v4.0.2

5. Root Cause Analysis

[kennytm]

while the extra spaces can be considered a bug, i don't think the digest text are supposed to be syntax-error-free (literals and lists are substituted by ? and ...).

[breezewish]

@kennytm Yes, we only expect syntax-error free for this specific case.

[ghost]

There are a couple of other issues with the digester if anyone wants to take a look at these together:
#14241
#18224

[bb7133]

Close this issue since it is duplicated with #30556.

[github-actions]

Please check whether the issue should be labeled with 'affects-x.y' or 'fixes-x.y.z', and then remove 'needs-more-info' label.