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]>