Keep the wall-clock check and exclude it from the distributed plan
Per the review on the PR: restore both test files to master and add only `SETTINGS make_distributed_plan = 0` to the `system.query_log` read. `SET make_distributed_plan = 1` at the top of the test is session-wide, so without the override that read is planned distributed as well. `EXPLAIN` shows `MergingAggregated (merge)` over `GatherExchange` and `ScatterExchange` above `ReadFromMergeTree (system.query_log)`; a background merge can then invalidate the coordinator's part snapshot and the read fails with `Code: 232 NO_SUCH_DATA_PART`. With the override the same read is a plain `Aggregating` over `ReadFromMergeTree`. This restores the wall-clock statement deleted by the previous commit, so the six `no-*` tags and the `.reference` line are master's again, and the only difference from master is the one settings clause. Verified on a master debug build that carries the finished-stage drain fix: 10 of 10 runs match the reference, max `query_duration_ms` 84 ms.
G
Groene AI committed
ae516bbb504ba97ded0d521b626cd2135eda99d2
Parent: 8a7fc53