From: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
---|---|
To: | Greg Smith <greg(at)2ndquadrant(dot)com> |
Cc: | Josh Berkus <josh(at)agliodbs(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Formatting Curmudgeons WAS: MMAP Buffers |
Date: | 2011-05-10 08:16:31 |
Message-ID: | 4DC8F45F.9020308@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 10.05.2011 04:43, Greg Smith wrote:
> Josh Berkus wrote:
>> As I don't think we can change this, I think the best answer is to
>> tell people
>> "Don't submit a big patch to PostgreSQL until you've done a few small
>> patches first. You'll regret it".
>
> When I last did a talk about getting started writing patches, I had a
> few people ask me afterwards if I'd ever run into problems with having
> patch submissions rejected. I said I hadn't. When asked what my secret
> was, I told them my first serious submission modified exactly one line
> of code[1]. And *that* I had to defend in regards to its performance
> impact.[2]
>
> Anyway, I think the intro message should be "Don't submit a big patch to
> PostgreSQL until you've done a small patch and some patch review"
> instead though.
Well, my first patch was two-phase commit. And I had never even used
PostgreSQL before I dived into the source tree and started to work on
that. I did, however, lurk on the pgsql-hackers mailing list for a few
months before posting, so I knew the social dynamics. I basically did
exactly what Robert described elsewhere in this thread, and successfully
avoided the culture shock.
--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2011-05-10 08:22:34 | Re: Process wakeups when idle and power consumption |
Previous Message | Leonardo Francalanci | 2011-05-10 07:35:21 | Re: switch UNLOGGED to LOGGED |