thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-12-19 12:52:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix crash in transactions with unsupported storage

Browse Source
This commit is contained in:
Raúl Marín 2024-12-10 16:24:53 +01:00
parent 787cf7db89
commit 173c409d3e
3 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Interpreters/InterpreterSelectQuery.cpp
View File

@ -588,7 +588,7 @@ InterpreterSelectQuery::InterpreterSelectQuery(
auto maybe_storage = DatabaseCatalog::instance().tryGetTable({table.table.database, table.table.table}, context);
if (!maybe_storage)
continue;
checkStorageSupportsTransactionsIfNeeded(storage, context, /* is_readonly_query */ true);
checkStorageSupportsTransactionsIfNeeded(maybe_storage, context, /* is_readonly_query */ true);
}
}

0
tests/queries/0_stateless/03282_memory_transaction_crash.reference Normal file
View File

9
tests/queries/0_stateless/03282_memory_transaction_crash.sql Normal file
View File

@ -0,0 +1,9 @@
-- Tags: zookeeper
-- https://github.com/ClickHouse/ClickHouse/issues/72887
DROP TABLE IF EXISTS t0;
CREATE TABLE t0 (c0 Int) ENGINE = Memory();
BEGIN TRANSACTION;
EXPLAIN SYNTAX SELECT 1 FROM (SELECT 1) tx JOIN t0 ON TRUE; -- { serverError NOT_IMPLEMENTED }
ROLLBACK;
DROP TABLE IF EXISTS t0;