Merge pull request ClickHouse#79851 from ClickHouse/fix-build-clang-20

Fix build on clang-20

Author: azat

src/Interpreters/InsertDependenciesBuilder.cpp

@@ -690,8 +690,9 @@ class ExecutingInnerQueryFromViewTransform final : public ExceptionKeepingTransf
 };
 
 
-InsertDependenciesBuilder::InsertDependenciesBuilder(StoragePtr table, ASTPtr query, Block insert_header,
-    bool async_insert_, bool skip_destination_table_, bool allow_materialized_,
+InsertDependenciesBuilder::InsertDependenciesBuilder(
+    StoragePtr table, ASTPtr query, Block insert_header,
+    bool async_insert_, bool skip_destination_table_,
     ContextPtr context)
     : init_table_id(table->getStorageID())
     , init_storage(table)
@@ -700,7 +701,6 @@ InsertDependenciesBuilder::InsertDependenciesBuilder(StoragePtr table, ASTPtr qu
     , init_context(context)
     , async_insert(async_insert_)
     , skip_destination_table(skip_destination_table_)
-    , allow_materialized(allow_materialized_)
     , views_error_registry(std::make_shared<ViewErrorsRegistry>())
     , logger(getLogger("InsertDependenciesBuilder"))
 {

src/Interpreters/InsertDependenciesBuilder.h

@@ -93,7 +93,7 @@ class InsertDependenciesBuilder : public std::enable_shared_from_this<InsertDepe
     void logQueryView(StorageID view_id, std::exception_ptr exception, bool before_start = false) const;
 
 protected:
-    InsertDependenciesBuilder(StoragePtr table, ASTPtr query, Block insert_header, bool async_insert_, bool skip_destination_table_, bool allow_materialized_, ContextPtr context);
+    InsertDependenciesBuilder(StoragePtr table, ASTPtr query, Block insert_header, bool async_insert_, bool skip_destination_table_, ContextPtr context);
 
 private:
     bool isView(StorageIDPrivate id) const;
@@ -117,7 +117,6 @@ class InsertDependenciesBuilder : public std::enable_shared_from_this<InsertDepe
 
     bool async_insert = false;
     bool skip_destination_table = false;
-    bool allow_materialized = false;
 
     StorageIDPrivate root_view;
 

src/Interpreters/InterpreterInsertQuery.cpp

@@ -450,7 +450,7 @@ QueryPipeline InterpreterInsertQuery::addInsertToSelectPipeline(ASTInsertQuery &
 
     auto insert_dependencies = InsertDependenciesBuilder::create(
         table, query_ptr, query_sample_block,
-        async_insert, /*skip_destination_table*/ no_destination, allow_materialized,
+        async_insert, /*skip_destination_table*/ no_destination,
         getContext());
 
     size_t sink_streams_size = table->supportsParallelInsert() ? max_insert_threads : 1;
@@ -632,7 +632,7 @@ QueryPipeline InterpreterInsertQuery::buildInsertPipeline(ASTInsertQuery & query
     // they are allowed to expose its virtuals columns to the dependent views
     auto insert_dependencies = InsertDependenciesBuilder::create(
         table, query_ptr, query_sample_block,
-        async_insert, /*skip_destination_table*/ no_destination, allow_materialized,
+        async_insert, /*skip_destination_table*/ no_destination,
         getContext());
 
     Chain chain = insert_dependencies->createChainWithDependencies();