Slow query response time when using version 42.2.21

[poirierlouis]

Describe the issue
When querying, the time took to respond is very long / slow comparing to older versions (42.2.20 / 42.2.6).
While using the driver through a web application using Play and Hibernate, response time goes from ~7 seconds to ~36 seconds.

Driver Version?
42.2.21

Java Version?
java version "1.8.0_291"
Java(TM) SE Runtime Environment (build 1.8.0_291-b10)
Java HotSpot(TM) 64-Bit Server VM (build 25.291-b10, mixed mode)

OS Version?
Microsoft Windows [version 10.0.19042.1052]

PostgreSQL Version?
PostgreSQL 11.5, compiled by Visual C++ build 1914, 64-bit

To Reproduce
Problem faced with a web application using Play v2.8.8 and Hibernate v5.5.0.
I plan to fill this issue with driver logs and minimal example when I'll have more time, likely tomorrow.

However, application code, database, OS, java version, etc. I've made sure that nothing changed.
I face this slow response time only when using 42.2.21. As soon as I use another version (42.2.20 or 42.2.6) response time goes back to normal.

Expected behavior
Driver performance should be at least as fast as previous versions.

Logs
Coming soon...

I submit this issue for now as I think this performance downside is quiet huge. It's a warning for others.

[davecramer]

Awesome, looking forward to the details.

[davecramer]

Strange, the is the second time I've closed an issue by mistake

[davecramer]

So I looked at the changes and the only possible change that might impact performance is 4fa2d5b and I can't see how...

[asaarnak]

Using 42.2.21 running our 300+ unit tests takes 3+ mins, using 42.2.20 it takes 1+ mins.

[mroloux]

We're having the same issue. Our builds slowed down a lot with 42.2.21. And even on production, query times went up.

[davecramer]

Can someone turn logging on and get some logs ? Or give me a repeatable test case

[davecramer]

@mroloux @asaarnak are you both using SSL connections ?

[mroloux]

Yes, I'm using SSL (on a Heroku database).

[davecramer]

Thanks, I know what the issue is. Fixing it may take some time, in the meantime downgrade to 42.2.20

[mroloux]

Thanks 👍

[davecramer]

So there is no "simple" solution to this and keeping 4fa2d5b I will have to revert this commit and release a new version.

[poirierlouis]

Thank you 👍
Sorry I couldn't provide a clear example. It's great that you found it quickly.

[davecramer]

before I release this, can you confirm that this https://oss.sonatype.org/content/repositories/orgpostgresql-1273/org/postgresql/postgresql/ fixes it ?

[poirierlouis]

Works just fine with this release on my environment 👌

[davecramer]

@poirierlouis thanks for testing it.

[MrEasy]

@davecramer quick test on my side also shows lower execution times again with .22 👍🏻

[davecramer]

OK, it's been released and pushed to maven central. Thanks for testing guys and apologies for the regression.