Fix comments

Author: EgorkaZ

ydb/core/kqp/gateway/kqp_metadata_loader.cpp

@@ -851,7 +851,7 @@ NThreading::TFuture<TTableMetadataResult> TKqpTableMetadataLoader::LoadTableMeta
                         {
                             auto externalDataSourceMetadata = result.GetValue();
                             auto newMetadata = EnrichExternalTable(externalTableMetadata, externalDataSourceMetadata);
-                            promise.SetValue(std::move(externalTableMetadata));
+                            promise.SetValue(std::move(newMetadata));
                         });
                         break;
                     }

ydb/core/kqp/provider/yql_kikimr_datasource.cpp

@@ -157,7 +157,8 @@ void ExtractReadAttributes(THashMap<std::pair<TString, TString>, THashMap<TStrin
 void GatherReadAttributes(THashMap<std::pair<TString, TString>, THashMap<TString, TString>>& result, const TExprNode& node, TExprContext& ctx) {
     if (node.IsCallable(ReadName)) {
         return ExtractReadAttributes(result, node, ctx);
-    } else if (node.IsCallable()) {
+    }
+    if (node.IsCallable()) {
         if (node.ChildrenSize() == 0) {
             return;
         }

ydb/core/kqp/provider/yql_kikimr_gateway.h

@@ -864,7 +864,7 @@ class IKikimrGateway : public TThrRefBase {
             return *this;
         }
 
-        NKikimr::NExternalSource::IExternalSourceFactory::TPtr ExternalSourceFactory = nullptr;
+        NKikimr::NExternalSource::IExternalSourceFactory::TPtr ExternalSourceFactory;
         THashMap<TString, TString> Options;
         bool RequestStats_ = false;
         bool WithPrivateTables_ = false;

ydb/library/yql/providers/s3/provider/yql_s3_datasource_type_ann.cpp

@@ -707,14 +707,6 @@ class TS3DataSourceTypeAnnotationTransformer : public TVisitorTransformerBase {
                     return true;
                 }
 
-                if (name == "infer") {
-                    TStringBuf data;
-                    if (!ExtractSettingValue(setting.Tail(), "infer"sv, format, {}, ctx, data)) {
-                        return false;
-                    }
-                    return true;
-                }
-
                 YQL_ENSURE(name == "projection"sv);
                 haveProjection = true;
                 if (!EnsureAtom(setting.Tail(), ctx)) {