public inbox for [email protected]  
help / color / mirror / Atom feed
From: Vladimir Sitnikov <[email protected]>
To: [email protected]
Subject: [pgjdbc/pgjdbc] c1a851: fix: avoid timezone conversions when sending Local...
Date: Tue, 05 Dec 2023 23:30:46 -0800
Message-ID: <pgjdbc/pgjdbc/push/refs/heads/master/[email protected]> (raw)

  Branch: refs/heads/master
  Home:   https://github.com/pgjdbc/pgjdbc
  Commit: c1a851c02b8690cda8e7b11a594f73ed27aa3189
      https://github.com/pgjdbc/pgjdbc/commit/c1a851c02b8690cda8e7b11a594f73ed27aa3189
  Author: Vladimir Sitnikov <[email protected]>
  Date:   2023-12-06 (Wed, 06 Dec 2023)

  Changed paths:
    M pgjdbc/src/main/java/org/postgresql/jdbc/PgResultSet.java
    M pgjdbc/src/main/java/org/postgresql/jdbc/TimestampUtils.java
    M pgjdbc/src/test/java/org/postgresql/test/jdbc42/SetObject310Test.java

  Log Message:
  -----------
  fix: avoid timezone conversions when sending LocalDateTime to the database

Previously, LocalDateTime was converted to OffsetDateTime using the default
timezone, which made certain values unrepresentable.
For instance, 2023-03-12T02:00 in America/New_York is hard to represent
as it undergoes DST change.

It does not change resultSet.getObject(int | string), and resultSet.getString(int | string)
and those methods would still have issues when accessing timestamps
like 2023-03-12T02:00 when the client time zone is America/New_York

Co-authored-by: Kevin Wooten <[email protected]>

Fixes https://github.com/pgjdbc/pgjdbc/issues/1390
Fixes https://github.com/pgjdbc/pgjdbc/issues/2850
Closes https://github.com/pgjdbc/pgjdbc/pull/1391








reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: [pgjdbc/pgjdbc] c1a851: fix: avoid timezone conversions when sending Local...
  In-Reply-To: <pgjdbc/pgjdbc/push/refs/heads/master/[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox