Alexander Kjäll

Results 38 comments of Alexander Kjäll

I don't understand why this fails on pg10. I will install pg10/java9 tomorrow and investigate in detail.

@marschall Thanks for the code review, I have implemented the suggested changes except `OffsetDateTime`. I looked around a bit, but I don't see how it should be implemented so that...

@marschall I don't have a complete picture of the situation, but I think that the implementation of `OffsetDateTime` needs to wait until #695 is fixed. Otherwise we risk having the...

@marschall I have added support for `OffsetDateTime`. I'm not 100% happy with how it turned out, since it only gives correct results when the Offset is UTC. But I agree...

@vlsi If you checkout the branch and change line 42 from `ZoneOffset.UTC` to `ZoneOffset.ofHours(2)` The test fails with the message: `java.lang.AssertionError: expected: but was:` I now realize that this might...

I investigated the time zone / UTC problem further, and the reason that it only works for UTC is that the column type `timestamp with time zone` doesn't store the...

Hi, I would just want to bump this, is there any further modification of this pull request that I should do?

@davecramer working on it, things have changed a bit since I did this PR :)

the test failure seems to be: ``` FAILURE 0.0sec, org.postgresql.replication.ReplicationConnectionTest > testConnectionNotValidWhenSessionTerminated java.lang.AssertionError: When postgresql terminate session with replication connection, isValid methos should return false, because next query on this...

Thanks for flagging this! This is extremely important when packaging for debian.