Brett Okken

Results 83 comments of Brett Okken

I think that linked issue was adding `OffsetTime` - and `OffsetDateTime` even predates it - but the overall point that it has been around for a long time remains. This...

If you are not using timestamptz, then use the [ResultSet getTimestamp](https://docs.oracle.com/en/java/javase/11/docs/api/java.sql/java/sql/ResultSet.html#getTimestamp(int,java.util.Calendar)) which takes a `Calendar`. Set the time zone on the calendar to use for interpreting the timestamp. As @davecramer...

OutOfMemory is an Error not an Exception. If the pgjdbc driver can be more resilient, that is great. However consumers should strongly consider the jvm option/flag to shutdown on OOME...

> I think the OOM is on the server. If that is correct then we should be able to recover from that. I think when he refers to "the server"...

I am pretty sure this is a result of the "rounding" logic: https://github.com/pgjdbc/pgjdbc/blob/master/pgjdbc/src/main/java/org/postgresql/jdbc/TimestampUtils.java#L804 LocalDateTime.MAX exceeds 499 nanos and is incremented by 1 micro, pushing to the next day. https://docs.oracle.com/javase/8/docs/api/java/time/LocalDateTime.html#MAX

With dates I have always found truncation in the face of mis-matched precision to be the least surprising behavior. My most obvious example is the old Oracle DATE type which...

This is may not be simple. There is nothing in the driver today which will translate into a primitive array. The current logic only supports reading into the object wrappers....

Can we start by defining in plain English what should define 2 PgArray instances as equal? Should it mean they represent the same logical contents? Should it mean the contain...

Should the existing UUID.[timestamp](https://docs.oracle.com/en/java/javase/25/docs/api//java.base/java/util/UUID.html#timestamp())() be updated to also support v7 uuid? If not, should there be another, similar method to extract the timestamp from a v7 UUID?

> while it may be confusing to use epochMillis and have timestamp throw UOE, it would be a hazard to do otherwise. +1 Should there be another method added to...