Commit 17adf80b authored by Thomas G. Lockhart's avatar Thomas G. Lockhart

Fix timestamp to date conversion for the case where timestamp uses a double

 precision storage format. Previously applied the same math as used for the
 64-bit integer storage format case, which was wrong.
 Problem introduced recently when the 64-bit storage format was
 implemented.
parent 606db06f
...@@ -8,7 +8,7 @@ ...@@ -8,7 +8,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/utils/adt/date.c,v 1.66 2002/04/21 19:48:12 thomas Exp $ * $Header: /cvsroot/pgsql/src/backend/utils/adt/date.c,v 1.67 2002/06/01 15:52:15 thomas Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
...@@ -308,22 +308,16 @@ timestamp_date(PG_FUNCTION_ARGS) ...@@ -308,22 +308,16 @@ timestamp_date(PG_FUNCTION_ARGS)
{ {
Timestamp timestamp = PG_GETARG_TIMESTAMP(0); Timestamp timestamp = PG_GETARG_TIMESTAMP(0);
DateADT result; DateADT result;
#if 0
struct tm tt,
*tm = &tt;
fsec_t fsec;
#endif
if (TIMESTAMP_NOT_FINITE(timestamp)) if (TIMESTAMP_NOT_FINITE(timestamp))
PG_RETURN_NULL(); PG_RETURN_NULL();
#if 0 #ifdef HAVE_INT64_TIMESTAMP
if (timestamp2tm(timestamp, NULL, tm, &fsec, NULL) != 0) /* Microseconds to days */
elog(ERROR, "Unable to convert timestamp to date");
result = date2j(tm->tm_year, tm->tm_mon, tm->tm_mday) - date2j(2000, 1, 1);
#else
result = (timestamp / INT64CONST(86400000000)); result = (timestamp / INT64CONST(86400000000));
#else
/* Seconds to days */
result = (timestamp / 86400.0);
#endif #endif
PG_RETURN_DATEADT(result); PG_RETURN_DATEADT(result);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment