Skip site navigation (1) Skip section navigation (2)

Re: Patch for JDBC fractional hour timezone offset bug

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Barry Lind <barry(at)xythos(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org, Thomas Lockhart <lockhart(at)alumni(dot)caltech(dot)edu>
Subject: Re: Patch for JDBC fractional hour timezone offset bug
Date: 2001-05-28 00:36:54
Message-ID: 200105280036.f4S0asn26329@candle.pha.pa.us (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-jdbcpgsql-patches
Thanks.  Applied.


> 
> Attached is a patch to fix the problem Thomas mentions below.  The JDBC 
> driver now correctly handles timezones that are offset fractional hours 
> from GMT (ie. -06:30).
> 
> thanks,
> --Barry
> 
> Thomas Lockhart wrote:
> 
> >> FYI, the reason I personally didn't want to apply this to 7.1.2 is
> >> because we already added two new bugs in 7.1.1, and I didn't want to be
> >> the one to add another.  Add that to the natural problems I have with
> >> JDBC, and I ran.  :-)
> > 
> > 
> > Understandable. You shouldn't *have* to take responsibility for every
> > patch coming in, so feel free to leave some things for others. And if
> > there is a problem with this one, I'll understand if you need to yell at
> > me a little ;)
> > 
> > As a (sort of) aside, by forcing me to inspect the code we have
> > uncovered a problem with atypical time zones which we would not
> > otherwise have noticed, at least until someone in Newfoundland or India
> > stumbled on it. No patch yet, but at least we know it is there...
> > 
> >                      - Thomas
> > 
> > ---------------------------(end of broadcast)---------------------------
> > TIP 6: Have you searched our list archives?
> > 
> > http://www.postgresql.org/search.mpl
> > 
> > 
> 

> *** ./org/postgresql/jdbc1/ResultSet.java.orig	Fri May 25 16:23:19 2001
> --- ./org/postgresql/jdbc1/ResultSet.java	Fri May 25 16:26:35 2001
> ***************
> *** 472,484 ****
>       //so this code strips off timezone info and adds on the GMT+/-...
>       //as well as adds a third digit for partial seconds if necessary
>       StringBuffer strBuf = new StringBuffer(s);
>       char sub = strBuf.charAt(strBuf.length()-3);
>       if (sub == '+' || sub == '-') {
>         strBuf.setLength(strBuf.length()-3);
>         if (subsecond)  {
> !         strBuf = strBuf.append('0').append("GMT").append(s.substring(s.length()-3, s.length())).append(":00");
>         } else {
> !         strBuf = strBuf.append("GMT").append(s.substring(s.length()-3, s.length())).append(":00");
>         }
>       } else if (subsecond) {
>         strBuf = strBuf.append('0');
> --- 472,506 ----
>       //so this code strips off timezone info and adds on the GMT+/-...
>       //as well as adds a third digit for partial seconds if necessary
>       StringBuffer strBuf = new StringBuffer(s);
> + 
> +     //we are looking to see if the backend has appended on a timezone.
> +     //currently postgresql will return +/-HH:MM or +/-HH for timezone offset
> +     //(i.e. -06, or +06:30, note the expectation of the leading zero for the
> +     //hours, and the use of the : for delimiter between hours and minutes)
> +     //if the backend ISO format changes in the future this code will
> +     //need to be changed as well
>       char sub = strBuf.charAt(strBuf.length()-3);
>       if (sub == '+' || sub == '-') {
>         strBuf.setLength(strBuf.length()-3);
>         if (subsecond)  {
> !         strBuf.append('0').append("GMT").append(s.substring(s.length()-3, s.length())).append(":00");
> !       } else {
> !         strBuf.append("GMT").append(s.substring(s.length()-3, s.length())).append(":00");
> !       }
> !     } else if (sub == ':') {
> !       //we may have found timezone info of format +/-HH:MM, or there is no
> !       //timezone info at all and this is the : preceding the seconds
> !       char sub2 = strBuf.charAt(strBuf.length()-5);
> !       if (sub2 == '+' || sub2 == '-') {
> !         //we have found timezone info of format +/-HH:MM
> !         strBuf.setLength(strBuf.length()-5);
> !         if (subsecond) {
> !           strBuf.append('0').append("GMT").append(s.substring(s.length()-5));
>           } else {
> !           strBuf.append("GMT").append(s.substring(s.length()-5));
> !         }
> !       } else if (subsecond) {
> !         strBuf.append('0');
>         }
>       } else if (subsecond) {
>         strBuf = strBuf.append('0');
> *** ./org/postgresql/jdbc2/ResultSet.java.orig	Fri May 25 16:23:34 2001
> --- ./org/postgresql/jdbc2/ResultSet.java	Fri May 25 16:24:25 2001
> ***************
> *** 484,497 ****
> --- 484,519 ----
>         sbuf.setLength(0);
>         sbuf.append(s);
>   
> +       //we are looking to see if the backend has appended on a timezone.
> +       //currently postgresql will return +/-HH:MM or +/-HH for timezone offset
> +       //(i.e. -06, or +06:30, note the expectation of the leading zero for the
> +       //hours, and the use of the : for delimiter between hours and minutes)
> +       //if the backend ISO format changes in the future this code will
> +       //need to be changed as well
>         char sub = sbuf.charAt(sbuf.length()-3);
>         if (sub == '+' || sub == '-') {
> +         //we have found timezone info of format +/-HH
>           sbuf.setLength(sbuf.length()-3);
>           if (subsecond)  {
>             sbuf.append('0').append("GMT").append(s.substring(s.length()-3)).append(":00");
>           } else {
>             sbuf.append("GMT").append(s.substring(s.length()-3)).append(":00");
>           }
> +       } else if (sub == ':') {
> +         //we may have found timezone info of format +/-HH:MM, or there is no
> +         //timezone info at all and this is the : preceding the seconds
> +         char sub2 = sbuf.charAt(sbuf.length()-5);
> +         if (sub2 == '+' || sub2 == '-') {
> +           //we have found timezone info of format +/-HH:MM
> +           sbuf.setLength(sbuf.length()-5);
> +           if (subsecond)  {
> +             sbuf.append('0').append("GMT").append(s.substring(s.length()-5));
> +           } else {
> +             sbuf.append("GMT").append(s.substring(s.length()-5));
> +           }
> +         } else if (subsecond) {
> +           sbuf.append('0');
> +         }
>         } else if (subsecond) {
>           sbuf.append('0');
>         }

> 
> ---------------------------(end of broadcast)---------------------------
> TIP 4: Don't 'kill -9' the postmaster

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman(at)candle(dot)pha(dot)pa(dot)us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

In response to

pgsql-hackers by date

Next:From: pgsql-interfacesDate: 2001-05-28 02:22:03
Subject: libpq++ in Apache Problem.
Previous:From: Bruce MomjianDate: 2001-05-28 00:15:09
Subject: Re: /contrib/earthdistance/Makefile patch

pgsql-patches by date

Next:From: Bruce MomjianDate: 2001-05-28 01:56:04
Subject: Re: A patch that adds primary key and unique key support to psql's \d
Previous:From: Alexander DedererDate: 2001-05-26 10:27:08
Subject: Patch for PGACCESS

pgsql-jdbc by date

Next:From: Bruce MomjianDate: 2001-05-28 13:22:01
Subject: Re: RE: [JDBC] ANT fix for jar creation
Previous:From: rjtalboDate: 2001-05-27 22:04:56
Subject: Re: [JDBC] Ant ???

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group