[stable-25-2-1] Fix overloaded & CTAS (#25308)

Author: maximyurchuk

ydb/core/kqp/executer_actor/kqp_scheme_executer.cpp

@@ -120,6 +120,11 @@ class TKqpSchemeExecuter : public TActorBootstrapped<TKqpSchemeExecuter> {
         auto* makeDir = modifyScheme->MutableMkDir();
         makeDir->SetName(GetSessionDirName());
 
+        NACLib::TDiffACL diffAcl;
+        diffAcl.SetInterruptInheritance(true);
+        auto* modifyAcl = modifyScheme->MutableModifyACL();
+        modifyAcl->SetDiffACL(diffAcl.SerializeAsString());
+
         auto promise = NewPromise<IKqpGateway::TGenericResult>();
         IActor* requestHandler = new TSchemeOpRequestHandler(ev.Release(), promise, false);
         RegisterWithSameMailbox(requestHandler);
@@ -162,6 +167,7 @@ class TKqpSchemeExecuter : public TActorBootstrapped<TKqpSchemeExecuter> {
                 NACLib::EAccessType::Allow,
                 access,
                 UserToken->GetUserSID());
+            diffAcl.SetInterruptInheritance(true);
             auto* modifyAcl = modifyScheme->MutableModifyACL();
             modifyAcl->SetDiffACL(diffAcl.SerializeAsString());
         }

ydb/core/kqp/host/kqp_statement_rewrite.cpp

@@ -264,7 +264,7 @@ namespace {
         const TString tmpTableName = TStringBuilder()
             << tableName
             << "_cas_"
-            << TAppData::RandomProvider->GenRand();
+            << TAppData::RandomProvider->GenUuid4().AsUuidString();
 
         const TString createTableName = (TStringBuilder()
                 << CanonizePath(sessionCtx->GetDatabase())

ydb/core/kqp/runtime/kqp_write_actor.cpp

@@ -609,9 +609,7 @@ class TKqpTableWriteActor : public TActorBootstrapped<TKqpTableWriteActor> {
                 << getIssues().ToOneLineString());
 
             const auto metadata = ShardedWriteController->GetMessageMetadata(ev->Get()->Record.GetOrigin());
-            YQL_ENSURE(metadata);
-
-            if (ev->Get()->Record.GetOverloadSubscribed() + 1 == metadata->NextOverloadSeqNo) {
+            if (metadata && ev->Get()->Record.GetOverloadSubscribed() + 1 == metadata->NextOverloadSeqNo) {
                 ResetShardRetries(ev->Get()->Record.GetOrigin(), ev->Cookie);
             }
 

ydb/core/tx/schemeshard/schemeshard_path.cpp

@@ -882,6 +882,12 @@ const TPath::TChecker& TPath::TChecker::FailOnRestrictedCreateInTempZone(bool al
     }
 
     if (allowCreateInTemporaryDir) {
+        if (std::all_of(Path.Elements.begin(), Path.Elements.end(),
+                [](const auto& element) { return !element->IsTemporary(); })) {
+            return Fail(status, TStringBuilder() << "path is not temporary"
+                << " (" << BasicPathInfo(Path.Base()) << ")"
+            );
+        }
         return *this;
     }