Hi, On Wed, Jul 22, 2009 at 2:20 AM, Robert Haas<robertmh...@gmail.com> wrote: > Fujii Masao, > > Are you planning to update this patch based on Martin's review?
Sure. Attached is an updated patch. > On Fri, Jul 17, 2009 at 5:26 PM, Martin Pihlak<martin.pih...@gmail.com> wrote: >> Here's my initial review of the non-blocking pqcomm patch. The patch applies >> cleanly and passes regression. Generally looks nice and clean. Couple of >> remarks >> from the department of nitpicking: Thanks for reviewing the patch! >> * In secure_poll() the handling of timeouts is different depending whether >> poll(), select() or SSL_pending() is used. The latter doesn't use the >> timeout value at all, and for select() it is impossible to specify >> indefinite >> timeout. Fixed. I tweaked the handling of the fifth argument 'timeout' of select(); when a negative number is specified to a timeout of secure_poll(), NULL is set to that 'timeout', which can block select() indefinitely. Since SSL_pending() doesn't wait for data to arrive (i.e., doesn't use timeout), I didn't change the code related to that function. >> * occasional "blank" lines consisting of a single tab character -- maybe >> a left-over from editor auto-indent. Not sure of how much a problem this >> is, given that the blanks will be removed by pg_indent. Fixed. >> * Comment on pq_wait() seems to have a typo: "-1 if an error directly." Fixed. Regards, -- Fujii Masao NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center
nonblocking_pqcomm_0722.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