On 27 January 2017 at 03:53, Robert Haas <robertmh...@gmail.com> wrote: > Sorry, this had slipped through the cracks -- I'm having a very hard > time keeping up with the flow of patches and emails. But it looks > good to me, except that it seems like CountDBBackends() needs the same > fix (and probably a corresponding documentation update).
Thanks for looking at this. Looks like there's a few other usages of CountDBBackends() which require background workers to be counted too, so I ended up creating CountDBConnections() as I didn't really think adding a bool flag to CountDBBackends was so nice. I thought about renaming CountUserBackends() to become CountUserConnections(), but I've not. Although, perhaps its better to break any third party stuff that uses that so that authors can review which behaviour they need rather than have their extension silently break? David -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
connection_limit_ignore_bgworkers_v2.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
