Jianfeng Jia has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/636

Change subject: Fix ASTERIXDB-1292
......................................................................

Fix ASTERIXDB-1292

Change-Id: Iee8e9b87145d9b3511874cd11d61ae87acdc94c7
---
M 
asterix-app/src/test/resources/runtimets/queries/temporal/parse_02/parse_02.3.query.aql
M 
asterix-app/src/test/resources/runtimets/results/temporal/parse_02/parse_02.1.adm
M 
asterix-om/src/main/java/org/apache/asterix/om/base/temporal/DateTimeFormatUtils.java
3 files changed, 7 insertions(+), 5 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/36/636/1

diff --git 
a/asterix-app/src/test/resources/runtimets/queries/temporal/parse_02/parse_02.3.query.aql
 
b/asterix-app/src/test/resources/runtimets/queries/temporal/parse_02/parse_02.3.query.aql
index dd1a2a5..318c4fe 100644
--- 
a/asterix-app/src/test/resources/runtimets/queries/temporal/parse_02/parse_02.3.query.aql
+++ 
b/asterix-app/src/test/resources/runtimets/queries/temporal/parse_02/parse_02.3.query.aql
@@ -26,7 +26,8 @@
 let $date := date("-0123-01-30")
 let $time := time("08:07:29.030Z")
 let $datetime := datetime("0137-12-31T23:59:59.999+08:00")
-return { 
+let $datetime2 := datetime("1000-10-10T10:10:10.100+00:00")
+return {
   "date-string-1": print-date($date, "YY/M/D"),
   "date-string-2": print-date($date, "MMM DD, YYYY"),
   "date-string-3": print-date($date, "YYYY/MM/DD"),
@@ -34,5 +35,6 @@
   "time-string-2": print-time($time, "hh.mm.ss.nnn a z"),
   "datetime-string-1": print-datetime($datetime, "MMM DD h:m:s.nnn a YY z"),
   "datetime-string-2": print-datetime($datetime, "YYYY/MMM/DD h:m:s.nnnz a"),
-  "datetime-string-3": print-datetime($datetime, "YYYY-MM-DDThh:mm:ss.nnnz")
+  "datetime-string-3": print-datetime($datetime, "YYYY-MM-DDThh:mm:ss.nnnz"),
+  "datetime-string-4": print-datetime($datetime2, "YYYY-MM-DDThh:mm:ss.nnnz")
  }
diff --git 
a/asterix-app/src/test/resources/runtimets/results/temporal/parse_02/parse_02.1.adm
 
b/asterix-app/src/test/resources/runtimets/results/temporal/parse_02/parse_02.1.adm
index 53035aa..68f22cc 100644
--- 
a/asterix-app/src/test/resources/runtimets/results/temporal/parse_02/parse_02.1.adm
+++ 
b/asterix-app/src/test/resources/runtimets/results/temporal/parse_02/parse_02.1.adm
@@ -1 +1 @@
-{ "date-string-1": "-123/1/30", "date-string-2": "JAN 30, -0123", 
"date-string-3": "-0123/01/30", "time-string-1": "8.7.29.03 AM Z", 
"time-string-2": "08.07.29.030 AM Z", "datetime-string-1": "DEC 31 3:59:59.999 
PM 137 Z", "datetime-string-2": "0137/DEC/31 3:59:59.999Z PM", 
"datetime-string-3": "0137-12-31T15:59:59.999Z" }
+{ "date-string-1": "-123/1/30", "date-string-2": "JAN 30, -0123", 
"date-string-3": "-0123/01/30", "time-string-1": "8.7.29.03 AM Z", 
"time-string-2": "08.07.29.030 AM Z", "datetime-string-1": "DEC 31 3:59:59.999 
PM 137 Z", "datetime-string-2": "0137/DEC/31 3:59:59.999Z PM", 
"datetime-string-3": "0137-12-31T15:59:59.999Z", "datetime-string-4": 
"1000-10-10T10:10:10.100Z" }
diff --git 
a/asterix-om/src/main/java/org/apache/asterix/om/base/temporal/DateTimeFormatUtils.java
 
b/asterix-om/src/main/java/org/apache/asterix/om/base/temporal/DateTimeFormatUtils.java
index 30ff44d..f3bd129 100644
--- 
a/asterix-om/src/main/java/org/apache/asterix/om/base/temporal/DateTimeFormatUtils.java
+++ 
b/asterix-om/src/main/java/org/apache/asterix/om/base/temporal/DateTimeFormatUtils.java
@@ -875,7 +875,7 @@
                         } else {
                             val = day;
                         }
-                        int valFieldCount = (int) Math.ceil(Math.log10(val));
+                        int valFieldCount = (int) Math.ceil(Math.log10(val + 
0.1));
                         if (val == 1 || val == 0) {
                             valFieldCount = 1;
                         }
@@ -904,7 +904,7 @@
                         appender.append(String.valueOf(val));
                         break;
                     case MILLISECOND:
-                        int msFieldCount = (int) Math.ceil(Math.log10(ms));
+                        int msFieldCount = (int) Math.ceil(Math.log10(ms + 
0.1));
                         for (int i = 0; i < 3 - msFieldCount; i++) {
                             appender.append('0');
                         }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/636
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iee8e9b87145d9b3511874cd11d61ae87acdc94c7
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Jianfeng Jia <[email protected]>

Reply via email to