Small fixes

Author: chenboy

src/optimizer/optimizer_task.cpp

@@ -95,8 +95,7 @@ void OptimizeExpression::execute() {
 
   std::sort(valid_rules.begin(), valid_rules.end());
   LOG_TRACE("OptimizeExpression::execute() op %d, valid rules : %lu",
-            static_cast<int>(group_expr_->Op().type()), valid_rules.size());
-
+            static_cast<int>(group_expr_->Op().GetType()), valid_rules.size());
   // Apply rule
   for (auto &r : valid_rules) {
     PushTask(new ApplyRule(group_expr_, r.rule, context_));

src/optimizer/plan_generator.cpp

@@ -458,7 +458,6 @@ void PlanGenerator::BuildAggregatePlan(
     const std::vector<std::shared_ptr<expression::AbstractExpression>>
         *groupby_cols,
     std::unique_ptr<expression::AbstractExpression> having_predicate) {
-  LOG_DEBUG("Generating Aggregate Plan");
   vector<planner::AggregatePlan::AggTerm> aggr_terms;
   vector<catalog::Column> output_schema_columns;
   DirectMapList dml;

test/sql/optimizer_sql_test.cpp

@@ -799,6 +799,7 @@ TEST_F(OptimizerSQLTests, NestedQueryWithAggregationTest) {
 }
 
 TEST_F(OptimizerSQLTests, NestedQueryInHavingTest) {
+  // 4 previously inserted tuples
   //  TestingSQLUtil::ExecuteSQLQuery("INSERT INTO test VALUES (1, 22, 333);");
   //  TestingSQLUtil::ExecuteSQLQuery("INSERT INTO test VALUES (2, 11, 000);");
   //  TestingSQLUtil::ExecuteSQLQuery("INSERT INTO test VALUES (3, 33, 444);");