git.postgresql.org Git - postgresql.git/commitdiff

git projects / postgresql.git / commitdiff

Make handling of INTERVAL DAY TO MINUTE and INTERVAL DAY TO SECOND input

2009年6月10日 05:05:03 +0000 (05:05 +0000)

more consistent with other cases, by having an unlabeled integer field
be treated as a number of minutes or seconds respectively. These cases
are outside the spec (which insists on full "dd hh:mm" or "dd hh:mm:ss"
input respectively), so it's not much help to us in deciding what to do.
But with this change, it's uniformly the case that an unlabeled integer
will be considered as being a number of the interval's rightmost field.
The change also takes us back to the 8.3 behavior of throwing error
for certain ambiguous inputs such as INTERVAL '1 2' DAY TO MINUTE.
Per recent discussion.

src/backend/utils/adt/datetime.c patch | blob | blame | history

src/test/regress/expected/interval.out patch | blob | blame | history

diff --git a/src/backend/utils/adt/datetime.c b/src/backend/utils/adt/datetime.c

index cf8009df9104817190a816873730a116ff9dfb65..e6f4689a59f9cd0d913ced7a5508bc183224c01b 100644 (file)

--- a/src/backend/utils/adt/datetime.c

+++ b/src/backend/utils/adt/datetime.c

@@ -8,7 +8,7 @@

* IDENTIFICATION

- * $PostgreSQL: pgsql/src/backend/utils/adt/datetime.c,v 1.206 2009年06月01日 16:55:11 tgl Exp $

+ * $PostgreSQL: pgsql/src/backend/utils/adt/datetime.c,v 1.207 2009年06月10日 05:05:03 tgl Exp $

*-------------------------------------------------------------------------

@@ -2917,17 +2917,17 @@ DecodeInterval(char **field, int *ftype, int nf, int range,

break;

case INTERVAL_MASK(HOUR):

case INTERVAL_MASK(DAY) | INTERVAL_MASK(HOUR):

- case INTERVAL_MASK(DAY) | INTERVAL_MASK(HOUR) | INTERVAL_MASK(MINUTE):

- case INTERVAL_MASK(DAY) | INTERVAL_MASK(HOUR) | INTERVAL_MASK(MINUTE) | INTERVAL_MASK(SECOND):

type = DTK_HOUR;

break;

case INTERVAL_MASK(MINUTE):

case INTERVAL_MASK(HOUR) | INTERVAL_MASK(MINUTE):

+ case INTERVAL_MASK(DAY) | INTERVAL_MASK(HOUR) | INTERVAL_MASK(MINUTE):

type = DTK_MINUTE;

break;

case INTERVAL_MASK(SECOND):

- case INTERVAL_MASK(HOUR) | INTERVAL_MASK(MINUTE) | INTERVAL_MASK(SECOND):

case INTERVAL_MASK(MINUTE) | INTERVAL_MASK(SECOND):

+ case INTERVAL_MASK(HOUR) | INTERVAL_MASK(MINUTE) | INTERVAL_MASK(SECOND):

+ case INTERVAL_MASK(DAY) | INTERVAL_MASK(HOUR) | INTERVAL_MASK(MINUTE) | INTERVAL_MASK(SECOND):

type = DTK_SECOND;

break;

default:

diff --git a/src/test/regress/expected/interval.out b/src/test/regress/expected/interval.out

index cf378867c73e0c42d84ee4f04c47dc710aaecc5f..a6f50b46221ffeac7ea170b1ab0a01ef5b36473d 100644 (file)

--- a/src/test/regress/expected/interval.out

+++ b/src/test/regress/expected/interval.out

@@ -466,11 +466,9 @@ SELECT interval '1 2:03:04' day to hour;

(1 row)

SELECT interval '1 2' day to minute;

- interval

-----------------

- 1 day 02:00:00

-(1 row)

+ERROR: invalid input syntax for type interval: "1 2"

+LINE 1: SELECT interval '1 2' day to minute;

+ ^

SELECT interval '1 2:03' day to minute;

interval

----------------

@@ -484,11 +482,9 @@ SELECT interval '1 2:03:04' day to minute;

(1 row)

SELECT interval '1 2' day to second;

- interval

-----------------

- 1 day 02:00:00

-(1 row)

+ERROR: invalid input syntax for type interval: "1 2"

+LINE 1: SELECT interval '1 2' day to second;

+ ^

SELECT interval '1 2:03' day to second;

interval

----------------

@@ -605,11 +601,9 @@ SELECT interval '1.234' second(2);

(1 row)

SELECT interval '1 2.345' day to second(2);

- interval

-----------------

- 1 day 02:20:42

-(1 row)

+ERROR: invalid input syntax for type interval: "1 2.345"

+LINE 1: SELECT interval '1 2.345' day to second(2);

+ ^

SELECT interval '1 2:03' day to second(2);

interval

----------------

This is the main PostgreSQL git repository.

RSS Atom