Merge pull request #8967 from CurtizJ/attach-if-not-exists

Fix query 'ATTACH TABLE IF NOT EXISTS'

dbms/src/Interpreters/InterpreterCreateQuery.cpp

@@ -545,10 +545,12 @@ BlockIO InterpreterCreateQuery::createTable(ASTCreateQuery & create)
     // If this is a stub ATTACH query, read the query definition from the database
     if (create.attach && !create.storage && !create.columns_list)
     {
+        bool if_not_exists = create.if_not_exists;
         // Table SQL definition is available even if the table is detached
         auto query = context.getDatabase(create.database)->getCreateTableQuery(context, create.table);
         create = query->as<ASTCreateQuery &>(); // Copy the saved create query, but use ATTACH instead of CREATE
         create.attach = true;
+        create.if_not_exists = if_not_exists;
     }
 
     String current_database = context.getCurrentDatabase();

dbms/tests/queries/0_stateless/01073_attach_if_not_exists.reference

+1

dbms/tests/queries/0_stateless/01073_attach_if_not_exists.sql

+CREATE TABLE t (a Int) ENGINE = Log;
+ATTACH TABLE t; -- { serverError 57 }
+ATTACH TABLE IF NOT EXISTS t;
+DETACH TABLE t;
+ATTACH TABLE IF NOT EXISTS t;
+EXISTS TABLE t;