Why was this code:

   #ifdef WIN32
       if (parallel_init_done && GetCurrentThreadId() != mainThreadId)
           ExitThread(code);
   #endif


removed from dumputils.c by commit 5e86c61a7eec0fdc6961493a150159fa8fc63b1c? The commit message doesn't refer to it at all.

I don't understand it at all, and there is nothing in Joachim's latest patch that would call ExitThread() from an exit handler, so the effect seems likely to me to break parallel windows processing in nasty ways if called from a worker thread, although I haven't tested it.

cheers

andrew

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to