On Sat, May 4, 2013 at 08:34:18PM +0900, Ian Lawrence Barwick wrote:
> 2013/4/24 Bruce Momjian <bruce@momjian.us>:
> > Thanks for the many suggestions on improving the 9.3 release notes.
> > There were many ideas I would have never thought of. Please keep the
> > suggestions coming.
>
> One small suggestion:
>
> <listitem>
> <para>
> Have <quote>session id</> (<literal>%c</>) in <link
> linkend="guc-log-line-prefix"><varname>log_line_prefix</></link>
> always output four hex digits after the period (Bruce Momjian)
> </para>
> </listitem>
>
> This doesn't sound quite right - on OS X at least, PIDs go up to
> 99998, which means
> %c may output 5 hex digits after the period. The following might be
Oh, I was curious if some OS had larger pid values. I am concerned you
aren't going to get session ids of consistent length on that platform.
> more pedantically
> accurate:
>
> <listitem>
> <para>
> Have <quote>session id</> (<literal>%c</>) in <link
> linkend="guc-log-line-prefix"><varname>log_line_prefix</></link>
> always pad the PID value with zeros so at least four hex digits are
> displayed after the period (Bruce Momjian)
> </para>
> </listitem>
OK, changed to:
Have <quote>session id</> (<literal>%c</>) in <link
linkend="guc-log-line-prefix"><varname>log_line_prefix</></link> always output at least four hex digits after the
period(Bruce Momjian)
This is such a minor change I am trying to keep it short.
> if my slightly disengaged brain is grokking the source correctly:
>
> src/backend/utils/error/elog.c:
> appendStringInfo(buf, "%lx.%04x", (long) (MyStartTime), MyProcPid);
Yep.
-- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB
http://enterprisedb.com
+ It's impossible for everything to be true. +