Address comments

Author: chenboy

src/binder/bind_node_visitor.cpp

@@ -75,9 +75,8 @@ void BindNodeVisitor::Visit(parser::SelectStatement *node) {
     }
     select_element->DeriveSubqueryFlag();
 
-    // Recursively deduce expression value type
+    // Traverse the expression to deduce expression value type and name
     select_element->DeduceExpressionType();
-    // Recursively deduce expression name
     select_element->DeduceExpressionName();
     new_select_list.push_back(std::move(select_element));
   }

src/include/expression/expression_util.h

@@ -428,7 +428,6 @@ class ExpressionUtil {
            ++tuple_idx) {
         if (child_expr->GetExpressionType() != ExpressionType::VALUE_TUPLE &&
             child_expr_maps[tuple_idx].count(child_expr)) {
-          // EvaluateExpression({child_expr_map}, child_expr);
           expr->SetChild(i, new TupleValueExpression(
                                 child_expr->GetValueType(), tuple_idx,
                                 child_expr_maps[tuple_idx][child_expr]));

src/optimizer/query_to_operator_transformer.cpp

@@ -406,6 +406,7 @@ bool QueryToOperatorTransformer::IsSupportedConjunctivePredicate(
   auto expr_type = expr->GetExpressionType();
   // Subquery with IN
   if (expr_type == ExpressionType::COMPARE_IN &&
+      expr->GetChild(0)->GetExpressionType() != ExpressionType::ROW_SUBQUERY &&
       expr->GetChild(1)->GetExpressionType() == ExpressionType::ROW_SUBQUERY) {
     return true;
   }