diff --git a/dbms/src/Processors/Executors/PipelineExecutor.h b/dbms/src/Processors/Executors/PipelineExecutor.h
index 20569d974ee..673151bd5eb 100644
--- a/dbms/src/Processors/Executors/PipelineExecutor.h
+++ b/dbms/src/Processors/Executors/PipelineExecutor.h
@@ -273,7 +273,6 @@ private:
     void executeSingleThread(size_t thread_num, size_t num_threads);
     void finish();
 
-public:
     String dumpPipeline() const;
 };
 
diff --git a/dbms/src/Processors/Formats/Impl/ConstantExpressionTemplate.cpp b/dbms/src/Processors/Formats/Impl/ConstantExpressionTemplate.cpp
index 31357b70ead..99369ffa26b 100644
--- a/dbms/src/Processors/Formats/Impl/ConstantExpressionTemplate.cpp
+++ b/dbms/src/Processors/Formats/Impl/ConstantExpressionTemplate.cpp
@@ -1,5 +1,4 @@
 #include <Columns/ColumnConst.h>
-#include <Common/typeid_cast.h>
 #include <DataTypes/DataTypesNumber.h>
 #include <DataTypes/DataTypeNullable.h>
 #include <DataTypes/FieldToDataType.h>
@@ -35,7 +34,7 @@ namespace ErrorCodes
 struct SpecialParserType
 {
     SpecialParserType() = default;
-    SpecialParserType(Field::Types::Which main_type_) : main_type(main_type_) {}
+    explicit SpecialParserType(Field::Types::Which main_type_) : main_type(main_type_) {}
 
     Field::Types::Which main_type = Field::Types::String;
     bool is_nullable = false;