You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iotdb.apache.org by qi...@apache.org on 2020/04/22 11:01:13 UTC
[incubator-iotdb] branch master updated: change to fullPath (#1087)
This is an automated email from the ASF dual-hosted git repository.
qiaojialin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-iotdb.git
The following commit(s) were added to refs/heads/master by this push:
new 726a8cf change to fullPath (#1087)
726a8cf is described below
commit 726a8cf13ac22fb80f171cd78e5015c81706ea4c
Author: Boris <zh...@gmail.com>
AuthorDate: Wed Apr 22 19:01:00 2020 +0800
change to fullPath (#1087)
---
server/src/main/antlr4/org/apache/iotdb/db/qp/strategy/SqlBase.g4 | 4 ++--
.../main/java/org/apache/iotdb/db/qp/strategy/LogicalGenerator.java | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/server/src/main/antlr4/org/apache/iotdb/db/qp/strategy/SqlBase.g4 b/server/src/main/antlr4/org/apache/iotdb/db/qp/strategy/SqlBase.g4
index 34fbee9..18f74bf 100644
--- a/server/src/main/antlr4/org/apache/iotdb/db/qp/strategy/SqlBase.g4
+++ b/server/src/main/antlr4/org/apache/iotdb/db/qp/strategy/SqlBase.g4
@@ -160,8 +160,8 @@ andExpression
;
predicate
- : (TIME | TIMESTAMP | suffixPath | prefixPath) comparisonOperator constant
- | (TIME | TIMESTAMP | suffixPath | prefixPath) inClause
+ : (TIME | TIMESTAMP | suffixPath | fullPath) comparisonOperator constant
+ | (TIME | TIMESTAMP | suffixPath | fullPath) inClause
| OPERATOR_NOT? LR_BRACKET orExpression RR_BRACKET
;
diff --git a/server/src/main/java/org/apache/iotdb/db/qp/strategy/LogicalGenerator.java b/server/src/main/java/org/apache/iotdb/db/qp/strategy/LogicalGenerator.java
index 330345a..f8bb52a 100644
--- a/server/src/main/java/org/apache/iotdb/db/qp/strategy/LogicalGenerator.java
+++ b/server/src/main/java/org/apache/iotdb/db/qp/strategy/LogicalGenerator.java
@@ -586,7 +586,7 @@ public class LogicalGenerator extends SqlBaseBaseListener {
throw new SQLParserException("group by fill doesn't support linear fill");
}
// all type use the same fill way
- if (SQLConstant.ALL.equals(typeClause.dataType().getText().toLowerCase())) {
+ if (SQLConstant.ALL.equalsIgnoreCase(typeClause.dataType().getText())) {
IFill fill;
if (typeClause.previousUntilLastClause() != null) {
fill = new PreviousFill(-1, true);
@@ -1147,8 +1147,8 @@ public class LogicalGenerator extends SqlBaseBaseListener {
if (ctx.TIME() != null || ctx.TIMESTAMP() != null) {
path = new Path(SQLConstant.RESERVED_TIME);
}
- if (ctx.prefixPath() != null) {
- path = parsePrefixPath(ctx.prefixPath());
+ if (ctx.fullPath() != null) {
+ path = parseFullPath(ctx.fullPath());
}
if (ctx.suffixPath() != null) {
path = parseSuffixPath(ctx.suffixPath());