github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[#4481] Added a comment explaining the rationale

Browse Source
This commit is contained in:
lukaseder 2015-08-20 11:54:01 +02:00
parent 25da0703a8
commit 9cd9867f89
2 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
jOOQ/src/main/java/org/jooq/impl/DateDiff.java
View File

@ -96,6 +96,9 @@ class DateDiff extends AbstractFunction<Integer> {
xx [/pro] */
case CUBRID:
case POSTGRES:
// [#4481] Parentheses are important in case this expression is
// placed in the context of other arithmetic
return field("({0} - {1})", getDataType(), date1, date2);
/* [pro] xx

3
jOOQ/src/main/java/org/jooq/impl/TimestampDiff.java
View File

@ -106,6 +106,9 @@ class TimestampDiff extends AbstractFunction<DayToSecond> {
xxxx xxxxxxx
xx [/pro] */
case POSTGRES:
// [#4481] Parentheses are important in case this expression is
// placed in the context of other arithmetic
return field("({0} - {1})", getDataType(), timestamp1, timestamp2);
// CUBRID's datetime operations operate on a millisecond level