xxq250
/
homework-jianmu
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Merge pull request #22673 from taosdata/fix/TS-3883 

fix: aggregate function omitted issue
This commit is contained in:
dapan1121 2023-09-01 11:01:58 +08:00 committed by GitHub
parent 371f6ab7f1 071e595344
commit 6a24e2cdc5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 29 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
source/libs/parser/src/parCalcConst.c
View File

@ -202,9 +202,25 @@ static int32_t calcConstProject(SNode* pProject, bool dual, SNode** pNew) {
return code;
}
typedef struct SIsUselessColCtx {
bool isUseless;
} SIsUselessColCtx ;
EDealRes checkUselessCol(SNode *pNode, void *pContext) {
SIsUselessColCtx *ctx = (SIsUselessColCtx *)pContext;
if (QUERY_NODE_FUNCTION == nodeType(pNode) && !fmIsScalarFunc(((SFunctionNode*)pNode)->funcId) &&
!fmIsPseudoColumnFunc(((SFunctionNode*)pNode)->funcId)) {
ctx->isUseless = false;
return DEAL_RES_END;
}
return DEAL_RES_CONTINUE;
}
static bool isUselessCol(SExprNode* pProj) {
if (QUERY_NODE_FUNCTION == nodeType(pProj) && !fmIsScalarFunc(((SFunctionNode*)pProj)->funcId) &&
!fmIsPseudoColumnFunc(((SFunctionNode*)pProj)->funcId)) {
SIsUselessColCtx ctx = {.isUseless = true};
nodesWalkExpr((SNode*)pProj, checkUselessCol, (void *)&ctx);
if (!ctx.isUseless) {
return false;
}
return NULL == ((SExprNode*)pProj)->pAssociation;

12
tests/system-test/2-query/nestedQuery.py
View File

@ -6082,8 +6082,18 @@ class TDTestCase:
tdLog.info(len(sql))
tdSql.error(sql)
#special sql
#6
tdSql.query("select 6-1 from stable_1;")
for i in range(self.fornum):
sql = "select count(*) from (select avg(q_int)/1000 from stable_1); "
tdLog.info(sql)
tdLog.info(len(sql))
tdSql.query(sql)
self.cur1.execute(sql)
self.explain_sql(sql)
#special sql
tdSql.query("select 7-1 from stable_1;")
for i in range(self.fornum):
sql = "select * from ( select _block_dist() from stable_1);"
tdSql.error(sql)