| From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
|---|---|
| To: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-committers(at)postgresql(dot)org |
| Subject: | Re: pgsql: * Stephen Frost (sfrost@snowman.net) wrote: |
| Date: | 2006-03-07 03:28:34 |
| Message-ID: | 440CFDE2.7050603@dunslane.net |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Bruce Momjian wrote:
>>Bruce, people would appreciate it if you made some effort to ensure
>>that commit messages describe the main purposes of the patch, rather
>>than being just the verbatim text of the last message in the thread.
>>The above log message is not merely overly verbose, but completely
>>useless. One might guess that the change had something to do with
>>Kerberos, but what?
>>
>>
>
>First I am hearing a complaint. I will try to clean them up.
>
>
>
Well, not everyone is as assertive as Tom :-)
One other thing: it's important that the first words in your commit
mesage summarise the contents of the commit, because that's what gets
put in the subject line of the committers email message. Recent examples
can be seen here:
http://archives.postgresql.org/pgsql-committers/2006-03/index.php
I don't read every committers message, and the subject lines can help me
filter them intelligently.
cheers
andrew
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2006-03-07 03:30:51 | Re: pgsql: * Stephen Frost (sfrost@snowman.net) wrote: |
| Previous Message | Bruce Momjian | 2006-03-07 03:03:09 | pgsql: Use SetConfigOption() to turn off "zero_damaged_pages" in |