Re: Small addition to PGInterval - Mailing list pgsql-jdbc

From Kris Jurka
Subject Re: Small addition to PGInterval
Date
Msg-id Pine.BSO.4.64.0704110347270.5256@leary.csoft.net
Whole thread Raw
In response to Small addition to PGInterval  (Hartmut Benz <hartmut.benz@ti-wmc.nl>)
Responses Re: Small addition to PGInterval  (Hartmut Benz <hartmut.benz@ti-wmc.nl>)
List pgsql-jdbc

On Thu, 29 Mar 2007, Hartmut Benz wrote:

> please find attached a small extension to PGInterval:
> - getTimeInMillis   - returns the number of milliseconds in the interval

getLengthInMillis doesn't account for months.

ONE_YEAR is seconds in a year instead of milliseconds.

Shouldn't we use the server's definition instead of google's for a year's
length in seconds:
select extract(epoch from '1 year'::interval);
  date_part
-----------
   31557600
(1 row)


> - copying constructor PGInterval(PGInterval) for type-safe, cast-free cloning
> - now implements Comaparable<PGInterval>

We can't use generics in this class because it must be buildable with
older JDK versions.

Also context diffs (-c) are preferred.

Kris Jurka

pgsql-jdbc by date:

Previous
From: Kris Jurka
Date:
Subject: Re: Exception in running query
Next
From: "Marc Mamin"
Date:
Subject: Re: Encoding issue (utf8): different strings received from java than from PGAdmin