Fix getting name of Object type and fix style check

Avogar · Avogar · commit 7ead747c059a · 2025-12-23T13:55:31.000Z
diff --git a/src/DataTypes/DataTypeObject.cpp b/src/DataTypes/DataTypeObject.cpp
@@ -24,6 +24,7 @@
 #include <Interpreters/Context.h>
 #include <Core/Settings.h>
 #include <IO/Operators.h>
+#include <boost/algorithm/string.hpp>
 
 #include "config.h"
 
@@ -195,7 +196,11 @@ String DataTypeObject::doGetName() const
     for (const auto & path : sorted_typed_paths)
     {
         write_separator();
-        out << backQuoteIfNeed(path) << " " << typed_paths.at(path)->getName();
+        /// We must quote path "SKIP" to avoid its confusion with SKIP keyword.
+        if (boost::to_upper_copy(path) == "SKIP")
+            out << backQuote(path) << " " << typed_paths.at(path)->getName();
+        else
+            out << backQuoteIfNeed(path) << " " << typed_paths.at(path)->getName();
     }
 
     std::vector<String> sorted_skip_paths;
diff --git a/src/Parsers/ASTObjectTypeArgument.cpp b/src/Parsers/ASTObjectTypeArgument.cpp
@@ -34,8 +34,6 @@ void ASTObjectTypedPathArgument::formatImpl(
     type->format(ostr, settings, state, frame);
 }
 
-
-
 ASTPtr ASTObjectTypeArgument::clone() const
 {
     auto res = std::make_shared<ASTObjectTypeArgument>(*this);

Original file line number	Diff line number	Diff line change
`@@ -34,8 +34,6 @@ void ASTObjectTypedPathArgument::formatImpl(`
`34`	`34`	`type->format(ostr, settings, state, frame);`
`35`	`35`	`}`
`36`	`36`
`37`		`-`
`38`		`-`
`39`	`37`	`ASTPtr ASTObjectTypeArgument::clone() const`
`40`	`38`	`{`
`41`	`39`	`auto res = std::make_shared<ASTObjectTypeArgument>(*this);`