Logical error: 'Inconsistent AST formatting between 'ExpressionList' and 'ExpressionList' in the query: (STID: 1941-1bfa)

[PedroTadim]

Important: This issue was automatically generated and is used by CI for matching failures. DO NOT modify the body content. DO NOT remove labels.

Test name: Logical error: 'Inconsistent AST formatting between 'ExpressionList' and 'ExpressionList' in the query: (STID: 1941-1bfa)
CI report: AST fuzzer (amd_debug, targeted)
Failing test history: cidb

Test output:

Error:
Logical error: 'Inconsistent AST formatting between 'ExpressionList' and 'ExpressionList' in the query:
RESTORE TABLE src_04218 AS dst_04218_ok FROM Memory(equals(or(2147483647, moduloOrZero((SELECT 2147483648, (SELECT toString(3, *, 65537) GROUP BY GROUPING SETS ((toNullable(materialize(2.)))) QUALIFY and(or(and(or(equals(toString(countArgMaxIf(toString('0.0.\0.0', assumeNotNull(toString(1048575, *, NULL), plus(toString(toString(toNullable(NULL), toString(toString(toInt256(-2147483647), *, 100), materialize(materialize(toLowCardinality('\0\0')))), *, '​')), isNull(toNullable(NULL)))), NULL, NULL), equals(b, 1048577), equals(toNullable(2147483646), b)) IGNORE NULLS, equals(a, assumeNotNull(assumeNotNull(toLowCardinality(-9223372036854775808)))), toFixedString(*, toString(NULL), NULL, isNotNull(NULL, 9223372036854775807))), Memory(toFixedString('10.24'))), exists((SELECT * FROM viewExplain('EXPLAIN', NULL, toString(NULL, toLowCardinality(256), -1, *), (SELECT DISTINCT i FROM index_append_test_test ANY LEFT JOIN (SELECT -inf, toNullable(assumeNotNull(2)) GROUP BY toNullable(toNullable(NULL, toString(toNullable('')))), greaterOrEquals(9223372036854775806, (SELECT DISTINCT NULL WHERE materialize((SELECT -0. GROUP BY 1)) LIMIT 1)), toString(toNullable(toFixedString(materialize(toFixedString('\0\0', 2)), 2)), toString(65537, *, -2147483649)), greater(7, 1048575, 256) QUALIFY substringUTF8(isNotNull(-1), divide(NULL, mid(NULL)))) AS alias2084 FINAL ON equals(alias2084.dst_04218_fail__fuzz_26, replicated_constraints2__fuzz_0) WHERE less(dst_04218_fail__fuzz_0, materialize(toLowCardinality(-2147483648)), moduloOrZero(toLowCardinality(100), b))))))), lessOrEquals(toInt32('0.0.0.0'), toString(0, NULL, *, 100))), equals('-214748364.7', countArgMaxState(10, toString(toString(*, not(greaterOrEquals(5, y), toString(materialize('\0\0'), toString(*, toLowCardinality(toInt128(257)), 2))), toNullable(NULL), '0.0.\0.0', assumeNotNull(assumeNotNull(9223372036854775807))))) AS alias2083))) LIMIT 151, 1024)), '00:00:00.000000000')), 257), '\0') SETTINGS allow_different_table_def = 1, backup_restore_keeper_max_retries = 1, backup_restore_keeper_retry_initial_backoff_ms = 1, backup_restore_keeper_retry_max_backoff_ms = 1 FORMAT `Null`
Formatted as:
RESTORE TABLE src_04218 AS dst_04218_ok FROM Memory(equals(or(2147483647, moduloOrZero((SELECT 2147483648, (SELECT toString(3, *, 65537) GROUP BY GROUPING SETS ((toNullable(materialize(2.)))) QUALIFY and(or(and(or(equals(toString(countArgMaxIf(toString('0.0.\0.0', assumeNotNull(toString(1048575, *, NULL), plus(toString(toString(toNullable(NULL), toString(toString(toInt256(-2147483647), *, 100), materialize(materialize(toLowCardinality('\0\0')))), *, '​')), isNull(toNullable(NULL)))), NULL, NULL), equals(b, 1048577), equals(toNullable(2147483646), b)) IGNORE NULLS, equals(a, assumeNotNull(assumeNotNull(toLowCardinality(-9223372036854775808)))), toFixedString(*, toString(NULL), NULL, isNotNull(NULL, 9223372036854775807))), Memory(toFixedString('10.24'))), exists((SELECT * FROM viewExplain('EXPLAIN', NULL, toString(NULL, toLowCardinality(256), -1, *), (SELECT DISTINCT i FROM index_append_test_test ANY LEFT JOIN (SELECT -inf, toNullable(assumeNotNull(2)) GROUP BY toNullable(toNullable(NULL, toString(toNullable('')))), greaterOrEquals(9223372036854775806, (SELECT DISTINCT NULL WHERE materialize((SELECT -0. GROUP BY 1)) LIMIT 1)), toString(toNullable(toFixedString(materialize(toFixedString('\0\0', 2)), 2)), toString(65537, *, -2147483649)), greater(7, 1048575, 256) QUALIFY substringUTF8(isNotNull(-1), divide(NULL, mid(NULL)))) AS alias2084 FINAL ON equals(alias2084.dst_04218_fail__fuzz_26, replicated_constraints2__fuzz_0) WHERE less(dst_04218_fail__fuzz_0, materialize(toLowCardinality(-2147483648)), moduloOrZero(toLowCardinality(100), b))))))), lessOrEquals(toInt32('0.0.0.0'), toString(0, NULL, *, 100))), equals('-214748364.7', countArgMaxState(10, toString(toString(*, not(tuple(greaterOrEquals(5, y), toString(materialize('\0\0'), toString(*, toLowCardinality(toInt128(257)), 2)))), toNullable(NULL), '0.0.\0.0', assumeNotNull(assumeNotNull(9223372036854775807))))) AS alias2083))) LIMIT 151, 1024)), '00:00:00.000000000')), 257), '\0') SETTINGS allow_different_table_def = 1, backup_restore_keeper_max_retries = 1, backup_restore_keeper_retry_initial_backoff_ms = 1, backup_restore_keeper_retry_max_backoff_ms = 1 FORMAT `Null`
Parsed and formatted back as:
RESTORE TABLE src_04218 AS dst_04218_ok FROM Memory(equals(or(2147483647, moduloOrZero((SELECT 2147483648, (SELECT toString(3, *, 65537) GROUP BY GROUPING SETS ((toNullable(materialize(2.)))) QUALIFY and(or(and(or(equals(toString(countArgMaxIf(toString('0.0.\0.0', assumeNotNull(toString(1048575, *, NULL), plus(toString(toString(toNullable(NULL), toString(toString(toInt256(-2147483647), *, 100), materialize(materialize(toLowCardinality('\0\0')))), *, '​')), isNull(toNullable(NULL)))), NULL, NULL), equals(b, 1048577), equals(toNullable(2147483646), b)) IGNORE NULLS, equals(a, assumeNotNull(assumeNotNull(toLowCardinality(-9223372036854775808)))), toFixedString(*, toString(NULL), NULL, isNotNull(NULL, 9223372036854775807))), Memory(toFixedString('10.24'))), exists((SELECT * FROM viewExplain('EXPLAIN', NULL, toString(NULL, toLowCardinality(256), -1, *), (SELECT DISTINCT i FROM index_append_test_test ANY LEFT JOIN (SELECT -inf, toNullable(assumeNotNull(2)) GROUP BY toNullable(toNullable(NULL, toString(toNullable('')))), greaterOrEquals(9223372036854775806, (SELECT DISTINCT NULL WHERE materialize((SELECT -0. GROUP BY 1)) LIMIT 1)), toString(toNullable(toFixedString(materialize(toFixedString('\0\0', 2)), 2)), toString(65537, *, -2147483649)), greater(7, 1048575, 256) QUALIFY substringUTF8(isNotNull(-1), divide(NULL, mid(NULL)))) AS alias2084 FINAL ON equals(alias2084.dst_04218_fail__fuzz_26, replicated_constraints2__fuzz_0) WHERE less(dst_04218_fail__fuzz_0, materialize(toLowCardinality(-2147483648)), moduloOrZero(toLowCardinality(100), b))))))), lessOrEquals(toInt32('0.0.0.0'), toString(0, NULL, *, 100))), equals('-214748364.7', countArgMaxState(10, toString(toString(*, not((tuple(greaterOrEquals(5, y), toString(materialize('\0\0'), toString(*, toLowCardinality(toInt128(257)), 2))))), toNullable(NULL), '0.0.\0.0', assumeNotNull(assumeNotNull(9223372036854775807))))) AS alias2083))) LIMIT 151, 1024)), '00:00:00.000000000')), 257), '\0') SETTINGS allow_different_table_def = 1, backup_restore_keeper_max_retries = 1, backup_restore_keeper_retry_initial_backoff_ms = 1, backup_restore_keeper_retry_max_backoff_ms = 1 FORMAT `Null`
Difference formatted as:
(greaterOrEquals(5, y), toString(materialize('\0\0'), toString(*, toLowCardinality(toInt128(257)), 2)))
ExpressionList, 0x00007ff2f618e320
-Function_tuple, 0x00007ff2f6110980
---

Stack trace:
__pthread_kill @ 0x00000000000969fd
raise @ 0x0000000000042476
__lgamma_r_finite @ 0x00000000000287f3
src/Common/Exception.cpp:60:5: DB::abortOnFailedAssertion(String const&, std::basic_string_view<char, std::char_traits<char>>, void* const*, unsigned long, unsigned long) @ 0x000000001409e576
src/Common/Exception.cpp:93:13: DB::handle_error_code(String const&, std::basic_string_view<char, std::char_traits<char>>, int, bool, std::vector<void*, std::allocator<void*>> const&) @ 0x000000001409f793
src/Common/Exception.cpp:146:19: DB::Exception::Exception(DB::Exception::MessageMasked&&, int, bool) @ 0x000000001409f959
src/Common/Exception.h:171:100: DB::Exception::Exception(String&&, int, String, bool) @ 0x000000000cde12d4
src/Common/Exception.h:57:54: DB::Exception::Exception(PreformattedMessage&&, int) @ 0x000000000cde0d66
src/Common/Exception.h:189:77: DB::Exception::Exception<String, String, std::basic_string_view<char, std::char_traits<char>>&, String&, String&, String, String, String, String>(int, FormatStringHelperImpl<std::type_identity<String>::type, std::type_identity<String>::type, std::type_identity<std::basic_string_view<char, std::char_traits<char>>&>::type, std::type_identity<String&>::type, std::type_identity<String&>::type, std::type_identity<String>::type, std::type_identity<String>::type, std::type_identity<String>::type, std::type_identity<String>::type>, String&&, String&&, std::basic_string_view<char, std::char_traits<char>>&, String&, String&, String&&, String&&, String&&, String&&) @ 0x0000000019b512b9
src/Interpreters/executeQuery.cpp:1310:31: DB::executeQueryImpl(char const*, char const*, std::shared_ptr<DB::Context>, DB::QueryFlags, DB::QueryProcessingStage::Enum, std::unique_ptr<DB::ReadBuffer, std::default_delete<DB::ReadBuffer>>&, boost::intrusive_ptr<DB::IAST>&, std::shared_ptr<DB::ImplicitTransactionControlExecutor>, std::function<void ()>, DB::QueryResultDetails&) @ 0x0000000019b451ad
src/Interpreters/executeQuery.cpp:2198:11: DB::executeQuery(String const&, std::shared_ptr<DB::Context>, DB::QueryFlags, DB::QueryProcessingStage::Enum) @ 0x0000000019b3bfa0
src/Server/TCPHandler.cpp:831:68: DB::TCPHandler::runImpl() @ 0x000000001eb7f458
src/Server/TCPHandler.cpp:3030:9: DB::TCPHandler::run() @ 0x000000001eb9ce64
base/poco/Net/src/TCPServerConnection.cpp:40:3: Poco::Net::TCPServerConnection::start() @ 0x0000000025f239ce
base/poco/Net/src/TCPServerDispatcher.cpp:115:42: Poco::Net::TCPServerDispatcher::run() @ 0x0000000025f23f12
base/poco/Foundation/src/ThreadPool.cpp:205:14: Poco::PooledThread::run() @ 0x0000000025eec83f
base/poco/Foundation/src/Thread_POSIX.cpp:341:27: Poco::ThreadImpl::runnableEntry(void*) @ 0x0000000025eeac0f
start_thread @ 0x0000000000094ac3
__clone3 @ 0x00000000001268d0

[PedroTadim]

@groeneai you can fix this as well

[rschu1ze]

@george-larionov Only FYI^^.

[groeneai]

Picking this up.

This is the same chronic STID 1941-1bfa family as #104605 (closed). The failure here is in the BACKUP_NAME kind of ASTFunction: Memory(...) in RESTORE TABLE ... FROM Memory(...) is parsed via parseBackupName in src/Parsers/ParserBackupQuery.cpp:220 and tagged as ASTFunction::Kind::BACKUP_NAME. ASTFunction::formatImpl at line 298 forces frame.allow_operators = false for CODEC / STATISTICS / BACKUP_NAME, and nothing ever resets it back to true, so the deeply nested not(a, b) inside the engine argument list inherits it. The descendant Function_tuple (lifted from the multi-arg not(a, b)) then falls back to its function-call form tuple(arg, arg, ...) and the standard re-parse paren round-trip diverges by one paren level. Same diff signature as the CODEC reproducers (Function_tuple vs ExpressionList at the diff site, parenthesized = true flag on re-parse).

Verified locally against a debug build:

• Minimal reproducer RESTORE TABLE src AS dst FROM Memory(not(materialize(1), materialize(2))) aborts the server with Logical error: 'Inconsistent AST formatting between 'ExpressionList' and 'ExpressionList' on master.
• The same query returns NUMBER_OF_ARGUMENTS_DOESNT_MATCH cleanly with the decideParensEmission fix from PR Fix inconsistent AST formatting for CODEC with tuple-wrapped multi-arg unary functions (STID 1941-1bfa) #104991 applied. The deeper CI-reduced shape (RESTORE TABLE ... FROM Memory(equals((SELECT not(x, y) FROM (SELECT 1 x, 2 y)), 1))) also round-trips cleanly.

So this is already covered by PR #104991 (the open follow-up fix for STID 1941-1bfa). The fix is gated on !frame.allow_operators and applies to all three BACKUP_NAME / CODEC / STATISTICS kinds at once. I have pushed an extra commit to that PR adding three explicit RESTORE TABLE / BACKUP TABLE regression cases plus the original failure shape from this issue, so the BACKUP_NAME path stays covered if either fix needs to be revised.

PR #104991 is awaiting review from @evillique / @alexey-milovidov / @george-larionov. Once it lands, master hits for this subvariant should stop. I will keep monitoring STID 1941-1bfa on master via CIDB.

cc @george-larionov (per @rschu1ze's earlier FYI)

Session: cron:clickhouse-ci-task-worker:20260520-174500