diff --git a/source/libs/parser/test/plannerTest.cpp b/source/libs/parser/test/plannerTest.cpp
index 5d3a6a42a62b8901812addbe1a75e0c7f3cae30e..89e41660702aee5ed3896b9ce5271c06251f8250 100644
--- a/source/libs/parser/test/plannerTest.cpp
+++ b/source/libs/parser/test/plannerTest.cpp
@@ -168,4 +168,14 @@ TEST(testCase, displayPlan) {
   generateLogicplan("select count(*) from `t.1abc` group by a");
   generateLogicplan("select count(*) from `t.1abc` interval(10s, 5s) sliding(7s)");
   generateLogicplan("select count(*),sum(a),avg(b),min(a+b) from `t.1abc`");
+
+  // order by + group by column + limit offset + fill
+
+
+  // join
+
+
+  // union
+
+
 }
\ No newline at end of file
diff --git a/source/libs/planner/src/planner.c b/source/libs/planner/src/planner.c
index 0621a9079859c1239063b1b9e2c0c1e28bf5ef48..00693892e980aed14380872fd79941ed030afc26 100644
--- a/source/libs/planner/src/planner.c
+++ b/source/libs/planner/src/planner.c
@@ -154,6 +154,8 @@ static SQueryPlanNode* createQueryNode(int32_t type, const char* name, SQueryPla
       memcpy(pNode->pExtInfo, pExtInfo, sizeof(SLimit));
       break;
     }
+    default:
+      assert(0);
   }
   
   return pNode;
@@ -257,14 +259,14 @@ static SQueryPlanNode* doCreateQueryPlanForOneTableImpl(SQueryStmtInfo* pQueryIn
     }
   }
 
+  // group by column not by tag
   if (numOfGroupCols != 0) {
     pNode = createQueryNode(QNODE_GROUPBY, "Groupby", &pNode, 1, NULL, 0, info, &pQueryInfo->groupbyExpr);
   }
 
   if (pQueryInfo->havingFieldNum > 0) {
 //    int32_t numOfExpr = (int32_t)taosArrayGetSize(pQueryInfo->exprList1);
-//    pNode = createQueryNode(QNODE_PROJECT, "Projection", &pNode, 1, pQueryInfo->exprList1->pData, numOfExpr, info,
-//        NULL);
+//    pNode = createQueryNode(QNODE_PROJECT, "Projection", &pNode, 1, pQueryInfo->exprList1->pData, numOfExpr, info, NULL);
   }
 
   if (pQueryInfo->fillType != TSDB_FILL_NONE) {