I wrote:
> Abhijit Menon-Sen <a...@2ndquadrant.com> writes:
>> At 2015-05-27 11:46:39 +0530, a...@2ndquadrant.com wrote:
>>> I'm trying a couple of approaches to that (e.g. using readdir directly
>>> instead of ReadDir), but other suggestions are welcome.
>> Here's what that looks like, but not yet fully tested.
> I doubt that that (not using AllocateDir) is a good idea in the backend,
Oh, scratch that. Reading the patch closer, I see you kept the use of
AllocateDir and only replaced ReadDir. That's kind of ugly (and certainly
requires a comment about the inconsistency) but it's probably ok from an
error handling standpoint. There are hard failure cases in AllocateDir,
but they seem unlikely to create problems in practice.
The other points stand though ...
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
