From 0812f794216a96adf60dc1fea6debd183dc64107 Mon Sep 17 00:00:00 2001
From: chertus <chertus@gmail.com>
Date: Wed, 26 Dec 2018 21:56:21 +0300
Subject: [PATCH] remove unused code

---
 dbms/src/Interpreters/ExpressionAnalyzer.cpp | 55 --------------------
 dbms/src/Interpreters/ExpressionAnalyzer.h   |  4 +-
 2 files changed, 1 insertion(+), 58 deletions(-)

diff --git a/dbms/src/Interpreters/ExpressionAnalyzer.cpp b/dbms/src/Interpreters/ExpressionAnalyzer.cpp
index e488aa5d81b..aa19a314634 100644
--- a/dbms/src/Interpreters/ExpressionAnalyzer.cpp
+++ b/dbms/src/Interpreters/ExpressionAnalyzer.cpp
@@ -337,56 +337,6 @@ void ExpressionAnalyzer::makeSetsForIndexImpl(const ASTPtr & node, const Block &
     }
 }
 
-bool ExpressionAnalyzer::isThereArrayJoin(const ASTPtr & ast)
-{
-    if (typeid_cast<ASTIdentifier *>(ast.get()))
-    {
-        return false;
-    }
-    else if (ASTFunction * node = typeid_cast<ASTFunction *>(ast.get()))
-    {
-        if (node->name == "arrayJoin")
-        {
-            return true;
-        }
-        if (functionIsInOrGlobalInOperator(node->name))
-        {
-            return isThereArrayJoin(node->arguments->children.at(0));
-        }
-        if (node->name == "indexHint")
-        {
-            return false;
-        }
-        if (AggregateFunctionFactory::instance().isAggregateFunctionName(node->name))
-        {
-            return false;
-        }
-        for (auto & child : node->arguments->children)
-        {
-            if (isThereArrayJoin(child))
-            {
-                return true;
-            }
-        }
-        return false;
-    }
-    else if (typeid_cast<ASTLiteral *>(ast.get()))
-    {
-        return false;
-    }
-    else
-    {
-        for (auto & child : ast->children)
-        {
-            if (isThereArrayJoin(child))
-            {
-                return true;
-            }
-        }
-        return false;
-    }
-}
-
 
 void ExpressionAnalyzer::getRootActions(const ASTPtr & ast, bool no_subqueries, ExpressionActionsPtr & actions, bool only_consts)
 {
@@ -1124,9 +1074,4 @@ void ExpressionAnalyzer::collectUsedColumns()
 }
 
 
-Names ExpressionAnalyzer::getRequiredSourceColumns() const
-{
-    return source_columns.getNames();
-}
-
 }
diff --git a/dbms/src/Interpreters/ExpressionAnalyzer.h b/dbms/src/Interpreters/ExpressionAnalyzer.h
index ba1e3a252d0..45944c48c85 100644
--- a/dbms/src/Interpreters/ExpressionAnalyzer.h
+++ b/dbms/src/Interpreters/ExpressionAnalyzer.h
@@ -150,7 +150,7 @@ public:
     /** Get a set of columns that are enough to read from the table to evaluate the expression.
       * Columns added from another table by JOIN are not counted.
       */
-    Names getRequiredSourceColumns() const;
+    Names getRequiredSourceColumns() const { return source_columns.getNames(); }
 
     /** These methods allow you to build a chain of transformations over a block, that receives values in the desired sections of the query.
       *
@@ -244,8 +244,6 @@ private:
 
     void addJoinAction(ExpressionActionsPtr & actions, bool only_types) const;
 
-    bool isThereArrayJoin(const ASTPtr & ast);
-
     /// If ast is ASTSelectQuery with JOIN, add actions for JOIN key columns.
     void getActionsFromJoinKeys(const ASTTableJoin & table_join, bool no_subqueries, ExpressionActionsPtr & actions);