db icon indicating copy to clipboard operation
db copied to clipboard

postgresql: add support for cancelation

Open Bogdanp opened this issue 3 years ago • 3 comments
trafficstars

This adds support for canceling Postgres queries according to 53.2.8. Canceling Requests in Progress. I don't love the connect&attach stuff, but I wanted to minimize the number of changes, and the ssl-context needs to be carried forward in case the server only accepts TLS connections. I don't feel strongly about adding the postgresql-cancel procedure, and would be happy with just the new cancel method on postgresql-connection<%> for now if you feel that eventually you might want to add a more general connection-cancel procedure that works on other connection types as well down the line.

Bogdanp avatar Sep 03 '22 19:09 Bogdanp

👋 ping

Bogdanp avatar Oct 04 '22 06:10 Bogdanp

Bump

Bogdanp avatar Dec 04 '22 07:12 Bogdanp

Hi @rmculpepper, any thoughts on this change?

Bogdanp avatar May 04 '24 14:05 Bogdanp

I've gone back and forth about the postgresql-cancel procedure, and at the moment I'd rather leave it as just a method. I'll merge this and to the update myself.

rmculpepper avatar Jun 03 '24 02:06 rmculpepper

Thanks!

Bogdanp avatar Jun 03 '24 06:06 Bogdanp