Re: Remove Windows crash dump support? - Mailing list pgsql-hackers

From Feng Tian
Subject Re: Remove Windows crash dump support?
Date
Msg-id CAFWGqnu9nimukNFuy2ZQYvWnfWP6XpAosawFpcpGHtDKDYmo+g@mail.gmail.com
Whole thread Raw
In response to Re: Remove Windows crash dump support?  (Craig Ringer <craig@2ndquadrant.com>)
List pgsql-hackers


On Wed, Dec 23, 2015 at 6:14 PM, Craig Ringer <craig@2ndquadrant.com> wrote:
On 22 December 2015 at 23:48, Alex Ignatov <a.ignatov@postgrespro.ru> wrote:
 
I think that you can debug crash dump since windbg exists. 

Nobody in their right mind uses windbg though. Visual Studio is really where it's at and the Express versions make it much more practical.

Just FYI.  Most developers of MS server (OS/SQL Server) team use windbg.  Windbg was my only debugger when I worked there and I never used Visual Studio.  


 

You can't even install Debugging Tools for Windows and Windbg standalone anymore.
 
Also I think that Postgres on Windows number  of instalations is so tiny  because people even today think that it is not so solid as unix version thats why you think that nobody use your code ;).

I disagree. Windows Pg users are often at bigger companies and don't talk about PostgreSQL as much. Often for fear of reprisals from other database vendors they have ongoing relationships with.  At least that's been my experience and I'm sure EDB folks will concur.
 
Today if my memory serves me right this code can not deal with buffer overflow. Am i right?

Stack overflow?

No, it can't. The stack has to be somewhat sane.
 
May be we need to add this functionality instead of drop support of it entirely

Why? I've never seen any sign anybody has used it, ever.

--
 Craig Ringer                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

pgsql-hackers by date:

Previous
From: Etsuro Fujita
Date:
Subject: Re: Minor code improvements to create_foreignscan_plan/ExecInitForeignScan
Next
From: Ioseph Kim
Date:
Subject: request patch pg_recvlogical.c, pg_receivexlog.c for nls