alaisi / postgres-async-driver

Asynchronous PostgreSQL Java driver
Apache License 2.0
287 stars 38 forks source link

introducing connection timeout and query read timeout #53

Closed jaroslawk closed 5 years ago

jaroslawk commented 6 years ago

Alaisi,

We are using your driver in our app under AWS cloud and we are missing some failover scenarios support:

I have provided you the tests for those scenarios.

Please let me know what do you think about those changes?

jaroslawk commented 6 years ago

@alaisi I am removing not important changes and also I see timeout was not working correctly. This time it covers also initial message - this covers our usage.

Let me know what you think.

I will add utest to prove it works as expected.

jaroslawk commented 6 years ago

@jaceksokol - good point. I have updated the code to take only connections which are alive under consideration.