Alvaro Herrera wrote: > Bruce Momjian wrote: > > > > I have applied a modified version of this patch. We now print the > > exception value in hex, and give a URL where the exception can be looked > > up. > > Humm, wouldn't it be more appropriate to put the URL in a errhint() > instead? > > > + ereport(lev, > > + > > + /*------ > > + translator: %s is a noun phrase describing a child process, > > such as > > + "server process" */ > > + (errmsg("%s (PID %d) was terminated by > > exception %X\nSee http://source.winehq.org/source/include/ntstatus.h for a > > description\nof the hex value.", > > + procname, pid, > > WTERMSIG(exitstatus)))); > > + #endif
Oops, forgot to mention that detail. We are using log_error() in one case, and ereport() in another. Let me do the hint in the report case, but I have to leave the log_error case alone because it takes only three arguments. -- Bruce Momjian [EMAIL PROTECTED] EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 9: In versions below 8.0, the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match