[iotdb] branch rel/1.2 updated: forbid group by tags query on view (#10238)

[qiaojialin]

This is an automated email from the ASF dual-hosted git repository.

qiaojialin pushed a commit to branch rel/1.2
in repository https://gitbox.apache.org/repos/asf/iotdb.git


The following commit(s) were added to refs/heads/rel/1.2 by this push:
     new cb817b77dd9 forbid group by tags query on view (#10238)
cb817b77dd9 is described below

commit cb817b77dd99ddf5913197ed8994dc511c24a120
Author: Marcos_Zyk <38524330+marcos...@users.noreply.github.com>
AuthorDate: Tue Jun 20 18:00:49 2023 +0800

    forbid group by tags query on view (#10238)
---
 .../org/apache/iotdb/db/mpp/plan/analyze/AnalyzeVisitor.java     | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git 
a/server/src/main/java/org/apache/iotdb/db/mpp/plan/analyze/AnalyzeVisitor.java 
b/server/src/main/java/org/apache/iotdb/db/mpp/plan/analyze/AnalyzeVisitor.java
index 94d6e0a483b..81256440c58 100644
--- 
a/server/src/main/java/org/apache/iotdb/db/mpp/plan/analyze/AnalyzeVisitor.java
+++ 
b/server/src/main/java/org/apache/iotdb/db/mpp/plan/analyze/AnalyzeVisitor.java
@@ -269,8 +269,13 @@ public class AnalyzeVisitor extends 
StatementVisitor<Analysis, MPPQueryContext>
 
         // make sure paths in logical view is fetched
         updateSchemaTreeByViews(analysis, schemaTree);
-        if (analysis.useLogicalView() && queryStatement.isAlignByDevice()) {
-          throw new SemanticException("Views cannot be used in ALIGN BY DEVICE 
query yet.");
+        if (analysis.useLogicalView()) {
+          if (queryStatement.isAlignByDevice()) {
+            throw new SemanticException("Views cannot be used in ALIGN BY 
DEVICE query yet.");
+          }
+          if (queryStatement.isGroupByTag()) {
+            throw new SemanticException("Views cannot be used in GROUP BY TAGS 
query yet.");
+          }
         }
       } finally {
         logger.debug("[EndFetchSchema]");