From d7b3b23fdfea0b445c9ef3c3c44c6a1dcc579183 Mon Sep 17 00:00:00 2001
From: jackwener <30525741+jackwener@users.noreply.github.com>
Date: Mon, 24 Jan 2022 19:56:24 +0800
Subject: [PATCH 1/5] parse date

---
 src/common/time/parser/datetime_parser.yy          |  8 ++++++++
 src/common/time/parser/test/DateTimeParserTest.cpp | 12 ++++++++++++
 2 files changed, 20 insertions(+)

diff --git a/src/common/time/parser/datetime_parser.yy b/src/common/time/parser/datetime_parser.yy
index f6d04d957e9..9a654a7ecec 100644
--- a/src/common/time/parser/datetime_parser.yy
+++ b/src/common/time/parser/datetime_parser.yy
@@ -131,6 +131,14 @@ date
       throw DatetimeParser::syntax_error(@1, result.toString());
     }
   }
+  | INTEGER {
+    $$ = new nebula::Date($1, 1, 1);
+    auto result = nebula::time::TimeUtils::validateDate(*$$);
+    if (!result.ok()) {
+      delete $$;
+      throw DatetimeParser::syntax_error(@1, result.toString());
+    }
+  }
   ;
 
 time
diff --git a/src/common/time/parser/test/DateTimeParserTest.cpp b/src/common/time/parser/test/DateTimeParserTest.cpp
index 939441b48b4..56b4c07676b 100644
--- a/src/common/time/parser/test/DateTimeParserTest.cpp
+++ b/src/common/time/parser/test/DateTimeParserTest.cpp
@@ -137,6 +137,13 @@ TEST(DatetimeReader, Date) {
     ASSERT_TRUE(result.ok()) << result.status();
     EXPECT_EQ(nebula::Date(2019, 1, 1), result.value());
   }
+  // lack month and day
+  {
+    auto parser = time::DatetimeReader::makeDateReader();
+    auto result = parser.readDate("2019");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::Date(2019, 1, 1), result.value());
+  }
   // TODO
   // lack month
   // {
@@ -182,6 +189,11 @@ TEST(DatetimeReader, DateFailed) {
     auto result = parser.readDate("2019-01-");
     EXPECT_FALSE(result.ok()) << result.value();
   }
+  {
+    auto parser = time::DatetimeReader::makeDateReader();
+    auto result = parser.readDate("2019-");
+    EXPECT_FALSE(result.ok()) << result.value();
+  }
   // not exits prefix
   {
     auto parser = time::DatetimeReader::makeDateReader();

From 7be1ffcc563d12aa4ceec7c794303b277492f09c Mon Sep 17 00:00:00 2001
From: jackwener <30525741+jackwener@users.noreply.github.com>
Date: Tue, 25 Jan 2022 15:08:58 +0800
Subject: [PATCH 2/5] enhance datetime

---
 src/common/time/parser/DatetimeReader.h       |  7 ++
 src/common/time/parser/datetime_parser.yy     | 53 +++++++++-----
 src/common/time/parser/datetime_scanner.lex   |  4 ++
 .../time/parser/test/DateTimeParserTest.cpp   | 70 +++++++++++++------
 4 files changed, 95 insertions(+), 39 deletions(-)

diff --git a/src/common/time/parser/DatetimeReader.h b/src/common/time/parser/DatetimeReader.h
index dfa8340f1f8..fda6f824301 100644
--- a/src/common/time/parser/DatetimeReader.h
+++ b/src/common/time/parser/DatetimeReader.h
@@ -33,16 +33,19 @@ class DatetimeReader {
   }
 
   StatusOr<DateTime> readDatetime(std::string input) {
+    input = kDatetimePrefix + input;
     return read(std::move(input));
   }
 
   StatusOr<Date> readDate(std::string input) {
+    input = kDatePrefix + input;
     auto result = read(std::move(input));
     NG_RETURN_IF_ERROR(result);
     return result.value().date();
   }
 
   StatusOr<Time> readTime(std::string input) {
+    input = kTimePrefix + input;
     auto result = read(std::move(input));
     NG_RETURN_IF_ERROR(result);
     return result.value().time();
@@ -60,6 +63,10 @@ class DatetimeReader {
   DatetimeParser parser_;
   std::string error_;
   DateTime *dt_{nullptr};
+
+  inline static const std::string kDatetimePrefix = "DATETIME__";
+  inline static const std::string kDatePrefix = "DATE__";
+  inline static const std::string kTimePrefix = "TIME__";
 };
 
 }  // namespace time
diff --git a/src/common/time/parser/datetime_parser.yy b/src/common/time/parser/datetime_parser.yy
index 9a654a7ecec..b7146bbcbf6 100644
--- a/src/common/time/parser/datetime_parser.yy
+++ b/src/common/time/parser/datetime_parser.yy
@@ -59,7 +59,7 @@ enum class Type {
 %token KW_TIME_ID
 
 /* symbols */
-%token TIME_DELIMITER SPACE POSITIVE NEGATIVE
+%token TIME_DELIMITER SPACE POSITIVE NEGATIVE DATETIMEPRFEIX DATEPRFEIX TIMEPRFEIX
 
 /* token type specification */
 %token <intVal> INTEGER
@@ -78,34 +78,43 @@ enum class Type {
 %%
 
 datetime
-  : date date_time_delimiter time opt_time_zone {
+  : DATETIMEPRFEIX date date_time_delimiter time opt_time_zone {
     if (outputType != nebula::time::Type::kDateTime) {
-      delete $1;
-      delete $3;
-      throw DatetimeParser::syntax_error(@1, "Mismatched date time type.");
+      delete $2;
+      delete $4;
+      throw DatetimeParser::syntax_error(@2, "Mismatched date time type.");
     }
-    $$ = new DateTime(TimeConversion::dateTimeShift(DateTime(*$1, *$3), -$4));
+    $$ = new DateTime(TimeConversion::dateTimeShift(DateTime(*$2, *$4), -$5));
     *output = $$;
-    delete $1;
-    delete $3;
+    delete $2;
+    delete $4;
   }
-  | date {
+  | DATETIMEPRFEIX date {
+    if (outputType != nebula::time::Type::kDateTime) {
+      delete $2;
+      throw DatetimeParser::syntax_error(@2, "Mismatched date time type.");
+    }
+    $$ = new DateTime(*$2);
+    *output = $$;
+    delete $2;
+  }
+  | DATEPRFEIX date {
     if (outputType != nebula::time::Type::kDate) {
-      delete $1;
-      throw DatetimeParser::syntax_error(@1, "Mismatched date time type.");
+      delete $2;
+      throw DatetimeParser::syntax_error(@2, "Mismatched date time type.");
     }
-    $$ = new DateTime(*$1);
+    $$ = new DateTime(*$2);
     *output = $$;
-    delete $1;
+    delete $2;
   }
-  | time opt_time_zone {
+  | TIMEPRFEIX time opt_time_zone {
     if (outputType != nebula::time::Type::kTime) {
-      delete $1;
-      throw DatetimeParser::syntax_error(@1, "Mismatched date time type.");
+      delete $2;
+      throw DatetimeParser::syntax_error(@2, "Mismatched date time type.");
     }
-    $$ = new DateTime(TimeConversion::dateTimeShift(DateTime(1970, 1, 1, $1->hour, $1->minute, $1->sec, $1->microsec), -$2));
+    $$ = new DateTime(TimeConversion::dateTimeShift(DateTime(1970, 1, 1, $2->hour, $2->minute, $2->sec, $2->microsec), -$3));
     *output = $$;
-    delete $1;
+    delete $2;
   }
   ;
 
@@ -168,6 +177,14 @@ time
       throw DatetimeParser::syntax_error(@1, result.toString());
     }
   }
+  | INTEGER {
+    $$ = new nebula::Time($1, 0, 0, 0);
+    auto result = nebula::time::TimeUtils::validateTime(*$$);
+    if (!result.ok()) {
+      delete $$;
+      throw DatetimeParser::syntax_error(@1, result.toString());
+    }
+  }
   ;
 
 opt_time_zone
diff --git a/src/common/time/parser/datetime_scanner.lex b/src/common/time/parser/datetime_scanner.lex
index 6df2f8321d4..85b11e05a8b 100644
--- a/src/common/time/parser/datetime_scanner.lex
+++ b/src/common/time/parser/datetime_scanner.lex
@@ -31,6 +31,10 @@ R_BRACKET                   "]"
 "+"                     { return TokenType::POSITIVE; }
 "-"                     { return TokenType::NEGATIVE; }
 
+"DATETIME__"            { return TokenType::DATETIMEPRFEIX; }
+"DATE__"                { return TokenType::DATEPRFEIX; }
+"TIME__"                { return TokenType::TIMEPRFEIX; }
+
 
 {DEC}+                  {
                           try {
diff --git a/src/common/time/parser/test/DateTimeParserTest.cpp b/src/common/time/parser/test/DateTimeParserTest.cpp
index 56b4c07676b..4b369024a1e 100644
--- a/src/common/time/parser/test/DateTimeParserTest.cpp
+++ b/src/common/time/parser/test/DateTimeParserTest.cpp
@@ -32,14 +32,25 @@ TEST(DatetimeReader, DateTime) {
     ASSERT_TRUE(result.ok()) << result.status();
     EXPECT_EQ(nebula::DateTime(2019, 1, 1, 22, 22, 3, 233300), result.value());
   }
-  // TODO
   // lack month
-  // {
-  // auto parser = time::DatetimeReader::makeDateTimeReader();
-  // auto result = parser.readDatetime("2019T22:22:3.2333");
-  // ASSERT_TRUE(result.ok()) << result.status();
-  // EXPECT_EQ(nebula::DateTime(2019, 1, 1, 22, 22, 3, 233300), result.value());
-  // }
+  {
+    auto parser = time::DatetimeReader::makeDateTimeReader();
+    auto result = parser.readDatetime("2019T22:22:3.2333");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::DateTime(2019, 1, 1, 22, 22, 3, 233300), result.value());
+  }
+  {
+    auto parser = time::DatetimeReader::makeDateTimeReader();
+    auto result = parser.readDatetime("2019T22:22");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::DateTime(2019, 1, 1, 22, 22, 0, 0), result.value());
+  }
+  {
+    auto parser = time::DatetimeReader::makeDateTimeReader();
+    auto result = parser.readDatetime("2019T22");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::DateTime(2019, 1, 1, 22, 0, 0, 0), result.value());
+  }
   // lack us
   {
     auto parser = time::DatetimeReader::makeDateTimeReader();
@@ -54,14 +65,32 @@ TEST(DatetimeReader, DateTime) {
     ASSERT_TRUE(result.ok()) << result.status();
     EXPECT_EQ(nebula::DateTime(2019, 1, 1, 22, 22, 0, 0), result.value());
   }
-  // TODO
   // lack minute
-  // {
-  // auto parser = time::DatetimeReader::makeDateTimeReader();
-  // auto result = parser.readDatetime("2019-1T22");
-  // ASSERT_TRUE(result.ok()) << result.status();
-  // EXPECT_EQ(nebula::DateTime(2019, 1, 1, 22, 0, 0, 0), result.value());
-  // }
+  {
+    auto parser = time::DatetimeReader::makeDateTimeReader();
+    auto result = parser.readDatetime("2019-1T22");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::DateTime(2019, 1, 1, 22, 0, 0, 0), result.value());
+  }
+  // datetime just include date
+  {
+    auto parser = time::DatetimeReader::makeDateTimeReader();
+    auto result = parser.readDatetime("2019-1-1");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::DateTime(2019, 1, 1, 0, 0, 0, 0), result.value());
+  }
+  {
+    auto parser = time::DatetimeReader::makeDateTimeReader();
+    auto result = parser.readDatetime("2019-1");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::DateTime(2019, 1, 1, 0, 0, 0, 0), result.value());
+  }
+  {
+    auto parser = time::DatetimeReader::makeDateTimeReader();
+    auto result = parser.readDatetime("2019");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::DateTime(2019, 1, 1, 0, 0, 0, 0), result.value());
+  }
 }
 
 TEST(DatetimeReader, DateTimeFailed) {
@@ -144,14 +173,13 @@ TEST(DatetimeReader, Date) {
     ASSERT_TRUE(result.ok()) << result.status();
     EXPECT_EQ(nebula::Date(2019, 1, 1), result.value());
   }
-  // TODO
   // lack month
-  // {
-  // auto parser = time::DatetimeReader::makeDateReader();
-  // auto result = parser.readDate("2019");
-  // ASSERT_TRUE(result.ok()) << result.status();
-  // EXPECT_EQ(nebula::Date(2019, 1, 1), result.value());
-  // }
+  {
+    auto parser = time::DatetimeReader::makeDateReader();
+    auto result = parser.readDate("2019");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::Date(2019, 1, 1), result.value());
+  }
 }
 
 TEST(DatetimeReader, DateFailed) {

From 303192f19e9eda5d5b0ee9d299fc096fca4db966 Mon Sep 17 00:00:00 2001
From: jackwener <30525741+jackwener@users.noreply.github.com>
Date: Tue, 25 Jan 2022 15:14:13 +0800
Subject: [PATCH 3/5] add test

---
 src/common/time/parser/test/DateTimeParserTest.cpp | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/src/common/time/parser/test/DateTimeParserTest.cpp b/src/common/time/parser/test/DateTimeParserTest.cpp
index 4b369024a1e..a51d39a9a72 100644
--- a/src/common/time/parser/test/DateTimeParserTest.cpp
+++ b/src/common/time/parser/test/DateTimeParserTest.cpp
@@ -264,14 +264,13 @@ TEST(DatetimeReader, Time) {
     ASSERT_TRUE(result.ok()) << result.status();
     EXPECT_EQ(nebula::Time(22, 22, 0, 0), result.value());
   }
-  // TODO
   // lack minute
-  // {
-  // auto parser = time::DatetimeReader::makeTimeReader();
-  // auto result = parser.readTime("22");
-  // ASSERT_TRUE(result.ok()) << result.status();
-  // EXPECT_EQ(nebula::Time(22, 0, 0, 0), result.value());
-  // }
+  {
+    auto parser = time::DatetimeReader::makeTimeReader();
+    auto result = parser.readTime("22");
+    ASSERT_TRUE(result.ok()) << result.status();
+    EXPECT_EQ(nebula::Time(22, 0, 0, 0), result.value());
+  }
 }
 
 TEST(DatetimeReader, TimeFailed) {

From c7f99db6161b957530e0619d44bff27f69927955 Mon Sep 17 00:00:00 2001
From: jackwener <30525741+jackwener@users.noreply.github.com>
Date: Tue, 25 Jan 2022 16:32:18 +0800
Subject: [PATCH 4/5] fix review

---
 src/common/time/parser/datetime_parser.yy | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/src/common/time/parser/datetime_parser.yy b/src/common/time/parser/datetime_parser.yy
index b7146bbcbf6..d45b5b29c9b 100644
--- a/src/common/time/parser/datetime_parser.yy
+++ b/src/common/time/parser/datetime_parser.yy
@@ -79,39 +79,22 @@ enum class Type {
 
 datetime
   : DATETIMEPRFEIX date date_time_delimiter time opt_time_zone {
-    if (outputType != nebula::time::Type::kDateTime) {
-      delete $2;
-      delete $4;
-      throw DatetimeParser::syntax_error(@2, "Mismatched date time type.");
-    }
     $$ = new DateTime(TimeConversion::dateTimeShift(DateTime(*$2, *$4), -$5));
     *output = $$;
     delete $2;
     delete $4;
   }
   | DATETIMEPRFEIX date {
-    if (outputType != nebula::time::Type::kDateTime) {
-      delete $2;
-      throw DatetimeParser::syntax_error(@2, "Mismatched date time type.");
-    }
     $$ = new DateTime(*$2);
     *output = $$;
     delete $2;
   }
   | DATEPRFEIX date {
-    if (outputType != nebula::time::Type::kDate) {
-      delete $2;
-      throw DatetimeParser::syntax_error(@2, "Mismatched date time type.");
-    }
     $$ = new DateTime(*$2);
     *output = $$;
     delete $2;
   }
   | TIMEPRFEIX time opt_time_zone {
-    if (outputType != nebula::time::Type::kTime) {
-      delete $2;
-      throw DatetimeParser::syntax_error(@2, "Mismatched date time type.");
-    }
     $$ = new DateTime(TimeConversion::dateTimeShift(DateTime(1970, 1, 1, $2->hour, $2->minute, $2->sec, $2->microsec), -$3));
     *output = $$;
     delete $2;

From dc87e44a881dcbe344466363c554f24d0bd6a8d7 Mon Sep 17 00:00:00 2001
From: jackwener <30525741+jackwener@users.noreply.github.com>
Date: Tue, 25 Jan 2022 16:47:08 +0800
Subject: [PATCH 5/5] rename

---
 src/common/time/parser/datetime_parser.yy   | 10 +++++-----
 src/common/time/parser/datetime_scanner.lex |  6 +++---
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/common/time/parser/datetime_parser.yy b/src/common/time/parser/datetime_parser.yy
index d45b5b29c9b..db054fec422 100644
--- a/src/common/time/parser/datetime_parser.yy
+++ b/src/common/time/parser/datetime_parser.yy
@@ -59,7 +59,7 @@ enum class Type {
 %token KW_TIME_ID
 
 /* symbols */
-%token TIME_DELIMITER SPACE POSITIVE NEGATIVE DATETIMEPRFEIX DATEPRFEIX TIMEPRFEIX
+%token TIME_DELIMITER SPACE POSITIVE NEGATIVE KW_DATETIME KW_DATE KW_TIME
 
 /* token type specification */
 %token <intVal> INTEGER
@@ -78,23 +78,23 @@ enum class Type {
 %%
 
 datetime
-  : DATETIMEPRFEIX date date_time_delimiter time opt_time_zone {
+  : KW_DATETIME date date_time_delimiter time opt_time_zone {
     $$ = new DateTime(TimeConversion::dateTimeShift(DateTime(*$2, *$4), -$5));
     *output = $$;
     delete $2;
     delete $4;
   }
-  | DATETIMEPRFEIX date {
+  | KW_DATETIME date {
     $$ = new DateTime(*$2);
     *output = $$;
     delete $2;
   }
-  | DATEPRFEIX date {
+  | KW_DATE date {
     $$ = new DateTime(*$2);
     *output = $$;
     delete $2;
   }
-  | TIMEPRFEIX time opt_time_zone {
+  | KW_TIME time opt_time_zone {
     $$ = new DateTime(TimeConversion::dateTimeShift(DateTime(1970, 1, 1, $2->hour, $2->minute, $2->sec, $2->microsec), -$3));
     *output = $$;
     delete $2;
diff --git a/src/common/time/parser/datetime_scanner.lex b/src/common/time/parser/datetime_scanner.lex
index 85b11e05a8b..a1fec9576dc 100644
--- a/src/common/time/parser/datetime_scanner.lex
+++ b/src/common/time/parser/datetime_scanner.lex
@@ -31,9 +31,9 @@ R_BRACKET                   "]"
 "+"                     { return TokenType::POSITIVE; }
 "-"                     { return TokenType::NEGATIVE; }
 
-"DATETIME__"            { return TokenType::DATETIMEPRFEIX; }
-"DATE__"                { return TokenType::DATEPRFEIX; }
-"TIME__"                { return TokenType::TIMEPRFEIX; }
+"DATETIME__"            { return TokenType::KW_DATETIME; }
+"DATE__"                { return TokenType::KW_DATE; }
+"TIME__"                { return TokenType::KW_TIME; }
 
 
 {DEC}+                  {