Thread: Re: Formatting Curmudgeons WAS: MMAP Buffers

Re: Formatting Curmudgeons WAS: MMAP Buffers

From
"Kevin Grittner"
Date:
Andrew Dunstan  wrote:
> What makes you think this isn't possible to run pgindent?
I have to say, I've been rather mystified by the difficulty
attributed to running pgindent.  During work on the SSI patch, I ran
it about once every two weeks on files involved in the patch, just so
that it would be easier to review by people used to that format.  I
also tried to keep src/tools/pgindent/typedefs.list up to date with
new structures, so that my runs were good.  Granted, when the
official run was done there were a few adjustments to typedefs.list,
and some comments which were added after the commit of the main part
of the patch hadn't yet been wrapped to the right line length, but on
the whole I didn't find it a big deal to stay relatively close by
doing periodic runs.  Maybe three minutes every two weeks.
When people talk like it's hugely difficult or hard to understand, I
wonder if they have actually made the attempt.  When someone is eager
for feedback on a patch, it doesn't seem unreasonable to me to ask
them to read the README for pgindent and try to generate a patch with
conforming results.
Now, the other aspect to this whole discussion is that people often
have code they have developed for academic purposes or for their own
use which they want to offer to the community "FWIW", and I think we
sometimes miss an opportunity to take advantage of someone else's
work because of an assumption that they have some vested interest in
it's acceptance.  The fact that someone doesn't care enough to try to
work with the community to get their patch accepted doesn't *always*
mean that we're better off for ignoring that patch.  Maybe that's
true 90% of the time or better, but it seems to me that sometimes our
community is a bit provincial.
And I can't help but wonder why, in an off-list discussion with
Michael Cahill about the SSI technology he commented that he was
originally intending to implement the technique in PostgreSQL, but
later chose Oracle Berkeley DB and then latter InnoDB instead. 
*Maybe* he was looking toward being hired by Oracle, and *maybe* it
was because the other databases already had predicate locking and
true serializable transaction isolation levels -- but was part of it
the reputation of the community?  I keep wondering.
-Kevin


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Sun, Apr 17, 2011 at 12:26 AM, Kevin Grittner
<Kevin.Grittner@wicourts.gov> wrote:
> Now, the other aspect to this whole discussion is that people often
> have code they have developed for academic purposes or for their own
> use which they want to offer to the community "FWIW", and I think we
> sometimes miss an opportunity to take advantage of someone else's
> work because of an assumption that they have some vested interest in
> it's acceptance.  The fact that someone doesn't care enough to try to
> work with the community to get their patch accepted doesn't *always*
> mean that we're better off for ignoring that patch.  Maybe that's
> true 90% of the time or better, but it seems to me that sometimes our
> community is a bit provincial.

We are.

On the other hand, cleaning up other people's not-ready-for-prime-time
patches isn't free.  If I spend 4 hours cleaning up a patch in
preparation for a commit, then that's 4 hours I don't get to spend on
my own work.  And since I *already* spend 3 or 4 times as much energy
on other people's work as I do on my own, I'm not willing to go much
further in that direction; if anything, I think I'd like to roll it
back a bit.  On the other hand, I am emphatically in favor of other
people who are not me being willing to do that kind of work; I think
it benefits our whole community, much as the work of people who write
their own patches or review or volunteer in any other way benefits our
whole community.

Because I commit approximately 10 patches per CommitFest, and review
perhaps another 5-10 that I don't end up committing (either because
they get rejected or because someone else commits them), the amount of
time that I can afford to spend on each of those patches is limited.
Generally, if I can't commit a normal-size patch in half an hour of
looking at it, I send back a review and move on.  For some patches
that I particularly care about, I have on occasion invested as much as
2-3 days (most recently, a big chunk of my Christmas vacation) to get
them beaten into shape for a commit.  I'd be happy to devote more time
per patch, but it ain't gonna happen as long as the number that I have
to handle to get the CommitFest finished on time remains in the
two-digit range.

That having been said, the kind of fixing up that you're talking about
*does* happen, when someone cares enough to make it happen.  We have
numerous examples in the archives where person A submits a patch, and
person B reviews it and, in lieu of a review, posts an updated patch,
sometimes when person A has meanwhile totally disappeared, or when
they haven't completely disappeared but don't have time to work on it.This is actually quite commonplace; it just
doesn'thappen for every 
patch.  It tends to happen only for the things someone is really
excited about because, well, fixing up someone else's bad code is not
one of life's great pleasures.  It'd be nice if we had even more of it
than we do, but this is an all-volunteer organization.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Dan Ports
Date:
On Sat, Apr 16, 2011 at 11:26:34PM -0500, Kevin Grittner wrote:
> I have to say, I've been rather mystified by the difficulty
> attributed to running pgindent.  During work on the SSI patch, I ran
> it about once every two weeks on files involved in the patch

Well, as a counterpoint: during work on the SSI patch, I did *not* run
pgindent. I attempted to, at one point, but was discouraged when I
realized that it required BSD indent and my Linux machine only had GNU
indent. That meant I would need to find, build, and install a new
version of indent, and keep it separate from my existing GNU indent.
Hardly impossible, but it's a lot more of a hassle than simply running a
script, and it left me wondering if I was going to run into other
issues even if I did get the right indent installed.

Andrew's instructions upthread would certainly have been helpful to
have in the pgindent README.

(To be fair, I would probably have made much more of an effort to run
pgindent if I didn't already know Kevin was running it periodically on
the SSI code.)


> And I can't help but wonder why, in an off-list discussion with
> Michael Cahill about the SSI technology he commented that he was
> originally intending to implement the technique in PostgreSQL, but
> later chose Oracle Berkeley DB and then latter InnoDB instead. 
> *Maybe* he was looking toward being hired by Oracle, and *maybe* it
> was because the other databases already had predicate locking and
> true serializable transaction isolation levels -- but was part of it
> the reputation of the community?  I keep wondering.

I would discount the first explanation (being hired at Oracle)
entirely. I think the second explanation is the correct one: it's
simply much more difficult to implement SSI atop a database that does
not already have predicate locking (as we know!)

But I am aware of other cases in which people in the academic community
have done work that could well be of interest to the Postgres community
but didn't submit their work here. In part, that was because they did
not have the time/motivation to get the work into a polished,
acceptable state, and in part because of the reputation of the
community. 

Dan

-- 
Dan R. K. Ports              MIT CSAIL                http://drkp.net/


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Dan Ports <drkp@csail.mit.edu> writes:
> ... But I am aware of other cases in which people in the academic community
> have done work that could well be of interest to the Postgres community
> but didn't submit their work here. In part, that was because they did
> not have the time/motivation to get the work into a polished,
> acceptable state, and in part because of the reputation of the
> community.

Well, if the author isn't interested in getting the work into a
committable state, it's not clear what's the point of submitting it.
It's not like people who are eager to do that kind of work on someone
else's patch are thick on the ground.

But I think the perception that we reject most patches is misplaced.
It's fairly easy to demonstrate that the default assumption around here
is that submitted patches will get committed.  Looking at the past five
commitfests (covering a bit more than a year), we committed 201 out of
305 patches, and only 10 were actually marked "rejected".  I'm too lazy
to try to determine just which of the 94 returned-with-feedback patches
got committed in later fests, but a quick scan suggests at least 20 did,
and there are more that might get committed in the next fest.  That puts
the overall patch acceptance rate at perhaps 75%.  At least since the CF
mechanism was instituted, it seems to me that the dynamic has been that
someone who doesn't like a patch has to show cause why it shouldn't get
committed, not the other way around.  Robert's recent comment that he
was afraid he'd have to spend time digging into the mmap patch to prove
it was broken reflects exactly that feeling.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Sun, Apr 17, 2011 at 7:13 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Dan Ports <drkp@csail.mit.edu> writes:
>> ... But I am aware of other cases in which people in the academic community
>> have done work that could well be of interest to the Postgres community
>> but didn't submit their work here. In part, that was because they did
>> not have the time/motivation to get the work into a polished,
>> acceptable state, and in part because of the reputation of the
>> community.
>
> Well, if the author isn't interested in getting the work into a
> committable state, it's not clear what's the point of submitting it.
> It's not like people who are eager to do that kind of work on someone
> else's patch are thick on the ground.
>
> But I think the perception that we reject most patches is misplaced.
> It's fairly easy to demonstrate that the default assumption around here
> is that submitted patches will get committed.  Looking at the past five
> commitfests (covering a bit more than a year), we committed 201 out of
> 305 patches, and only 10 were actually marked "rejected".  I'm too lazy
> to try to determine just which of the 94 returned-with-feedback patches
> got committed in later fests, but a quick scan suggests at least 20 did,
> and there are more that might get committed in the next fest.  That puts
> the overall patch acceptance rate at perhaps 75%.

That someone overstates the acceptance rate, because it ignores the
patches that people post and immediately get flamed to a well-done
crisp before adding them to the CF app, but there are not very many of
those any more.  (If someone thinks I'm wrong about this, they are
cheerfully invited to provide the evidence.  It is certainly possible
that I'm guilty of selective memory; this is just how I remember it.)

> At least since the CF
> mechanism was instituted, it seems to me that the dynamic has been that
> someone who doesn't like a patch has to show cause why it shouldn't get
> committed, not the other way around.  Robert's recent comment that he
> was afraid he'd have to spend time digging into the mmap patch to prove
> it was broken reflects exactly that feeling.

Yes, and I think it's also telling that the response to that was not
"oh, gee, if Robert thinks this patch is totally busted, we'd better
take that concern seriously" but rather "stop picking on the guy who
submitted the patch".  Maybe someone out there is under the impression
that I get high off of rejecting patches; but the statistics you cite
from the CF app don't exactly support the contention that I'm going
around looking for reasons to reject things, or if I am, I'm doing a
pretty terrible job finding them.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andrew Dunstan
Date:

On 04/17/2011 07:41 PM, Robert Haas wrote:
>
>> That puts the overall patch acceptance rate at perhaps 75%.
> That someone overstates the acceptance rate, because it ignores the
> patches that people post and immediately get flamed to a well-done
> crisp before adding them to the CF app, but there are not very many of
> those any more.
>

I don't believe there were ever terribly many of them.

cheers

andrew


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Andrew Dunstan <andrew@dunslane.net> writes:
> On 04/17/2011 07:41 PM, Robert Haas wrote:
>>> That puts the overall patch acceptance rate at perhaps 75%.

>> That someone overstates the acceptance rate, because it ignores the
>> patches that people post and immediately get flamed to a well-done
>> crisp before adding them to the CF app, but there are not very many of
>> those any more.

> I don't believe there were ever terribly many of them.

Well, that number also ignores patches that were *committed* without
ever making it to the CF list.  There aren't terribly many of those
either I think, but it does happen, particularly for small patches.
If you want to argue about the acceptance rate for out-of-CF-process
patches you'd have to do some serious digging in the archives to say
anything about what it is.

But anyway this is quibbling.  The point I was trying to make is that
our patch acceptance rate is fairly far north of 50%, not south of it.
So we might hold people's feet to the fire a bit in the process, but
it's hardly impossible to get a patch committed.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Robert Haas <robertmhaas@gmail.com> writes:
> ... Maybe someone out there is under the impression
> that I get high off of rejecting patches; but the statistics you cite
> from the CF app don't exactly support the contention that I'm going
> around looking for reasons to reject things, or if I am, I'm doing a
> pretty terrible job finding them.

Hm ... there are people out there who think *I* get high off rejecting
patches.  I have a t-shirt to prove it.  But I seem to be pretty
ineffective at it too, judging from these numbers.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Alvaro Herrera
Date:
Excerpts from Tom Lane's message of lun abr 18 02:50:22 -0300 2011:
> Robert Haas <robertmhaas@gmail.com> writes:
> > ... Maybe someone out there is under the impression
> > that I get high off of rejecting patches; but the statistics you cite
> > from the CF app don't exactly support the contention that I'm going
> > around looking for reasons to reject things, or if I am, I'm doing a
> > pretty terrible job finding them.
> 
> Hm ... there are people out there who think *I* get high off rejecting
> patches.  I have a t-shirt to prove it.  But I seem to be pretty
> ineffective at it too, judging from these numbers.

Does this mean we need an auction to get Robert a nice $1000 t-shirt?

-- 
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Joshua Berkus
Date:
Robert, Tom,

> Hm ... there are people out there who think *I* get high off rejecting
> patches. I have a t-shirt to prove it. But I seem to be pretty
> ineffective at it too, judging from these numbers.

It's a question of how we reject patches, especially first-time patches.   We can reject them in a way which makes the
submittermore likely to fix them and/or work on something else, or we can reject them in a way which discourages people
fromsubmitting to PostgreSQL at all.
 

For example, the emails to Radoslaw mentioned nothing about pg_ident, documented spacing requirements, accidental
inclusionof files he didn't mean to touch, etc.  Instead, a couple of people told him he should abandon his chosen
developmentIDE in favor of emacs or vim.  Radoslaw happens to be thick-skinned and persistent, but other first-time
submitterswould have given up at that point and run off to a more welcoming project.
 

Mind, even better would be to get our "so you're submitting a patch" documentation and tools into shape; that way, all
weneed to do is send the first-time submitter a link.  Will work on that between testing ...
 

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com
San Francisco


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Josh Berkus
Date:
On 4/18/11 10:57 AM, Robert Haas wrote:
> So first of all, no it's not fixable with sed.  But secondly, writing
> "*please*" here seems to evince a level of frustration which is
> entirely out of proportion to the really rather mild comments which
> preceded it.  What made you write it that way?

I'll admit that the conversation I'd had at the Drizzle BOF the previous
night strongly influenced me.

> to this continual commentary that we
> are not nice enough to people, especially newcomers.  Well, OK, maybe
> we're not.  But you know what?  We're trying really hard, and getting
> accused of being nasty when we actually weren't is kind of a tough
> pill to swallow ... But
> in this case I think you were too quick off the trigger

Well, my apologies to you.  You are probably correct.

In any case, I think the answer to this is constructive; better
documentation and tools to let submitters get their code into good shape
in the first place so that we don't have discussions about formatting.
That way we waste *neither* the reviewers' nor the submitters' time.

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Josh Berkus
Date:

> Does this mean we need an auction to get Robert a nice $1000 t-shirt?

... starting hunting through Robert's emails for a good quote ...


-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Mon, Apr 18, 2011 at 12:52 PM, Joshua Berkus <josh@agliodbs.com> wrote:
> Robert, Tom,
>
>> Hm ... there are people out there who think *I* get high off rejecting
>> patches. I have a t-shirt to prove it. But I seem to be pretty
>> ineffective at it too, judging from these numbers.
>
> It's a question of how we reject patches, especially first-time patches.   We can reject them in a way which makes
thesubmitter more likely to fix them and/or work on something else, or we can reject them in a way which discourages
peoplefrom submitting to PostgreSQL at all. 
>
> For example, the emails to Radoslaw mentioned nothing about pg_ident, documented spacing requirements, accidental
inclusionof files he didn't mean to touch, etc.  Instead, a couple of people told him he should abandon his chosen
developmentIDE in favor of emacs or vim.  Radoslaw happens to be thick-skinned and persistent, but other first-time
submitterswould have given up at that point and run off to a more welcoming project. 

Actually, the first reply was a very polite reply from Heikki pointing
out the problem very gently and asking for a theory of operation.

Radoslaw replied and said that he understood the formatting problem,
but his editor was mangling it:

>> Yes, but, hmm... in Netbeans I had really long gaps (probably 8 spaces, from tabs), so deeper "ifs", comments at the
andof variables, went of out my screen. I really wanted to not format this, but sometimes I needed. 

That prompted one - ONE! - person to reply and suggest that the use of
another editor might work better.   At which point, we got an
apparently-exasperated note from you suggesting that a 10% performance
improvement wasn't enough (which I disagree with) and that it was
wrong for people to worry about whether they could read the patch well
enough to understand it (which I also disagree with).  Conceding that
some of the following discussion may have gotten a little harsh
(though frankly I think that was mostly directed at your remark, not
the OP), what prompted that original note?  Here it is:

>> Guys, can we *please* focus on the patch for now, rather than the formatting, which is fixable with sed?

So first of all, no it's not fixable with sed.  But secondly, writing
"*please*" here seems to evince a level of frustration which is
entirely out of proportion to the really rather mild comments which
preceded it.  What made you write it that way?

I think that the harshness of the reaction to your statement is a
reflection of some underlying frustration on my part and perhaps also
on the part of other reviewers - to this continual commentary that we
are not nice enough to people, especially newcomers.  Well, OK, maybe
we're not.  But you know what?  We're trying really hard, and getting
accused of being nasty when we actually weren't is kind of a tough
pill to swallow.  I would really like to see someone go back and look
at every patch from a newcomer that's been submitted in the last year,
and rate the reaction to that patch on an A-F scale.  Then let's have
a discussion about what percentage we did well on, and what percentage
we did poorly on, and how we could have done better.  When we actually
start raking someone over the coals, I think it's great and a helpful
service for you to jump in and say - hold on a minute, timeout.  But
in this case I think you were too quick off the trigger, and I don't
think that acting as if it's unreasonable to want a patch that
conforms to our submission guidelines is doing anyone any favors.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Mon, Apr 18, 2011 at 3:17 PM, Josh Berkus <josh@agliodbs.com> wrote:
> In any case, I think the answer to this is constructive; better
> documentation and tools to let submitters get their code into good shape
> in the first place so that we don't have discussions about formatting.
> That way we waste *neither* the reviewers' nor the submitters' time.

Well, I'm all in favor of better documentation, but I think the
biggest thing we need to do is get the word out:

1. We realize we have been too trigger-happy sometimes.
2. But we really want you to participate.
3. And we are trying very hard to do better.
4. And please tell us if we screw up, so we can keep working on it.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Josh Berkus
Date:
Robert,

> 1. We realize we have been too trigger-happy sometimes.
> 2. But we really want you to participate.
> 3. And we are trying very hard to do better.
> 4. And please tell us if we screw up, so we can keep working on it.

I received a private offlist email from someone who didn't feel
comfortable bringing up their issues with this list publicly.  Let me
quote from it, because I think it pins part of the issue:

"I believe this is due to the current postgresql "commitfest" process
whereby there is no real way to present new ideas or technologies
without coming to the table with a fully-baked plan and patch. This is
obvious even in the name "commitfest" since the expectation is that
every patch presented is considered ready-to-commit by the patch
presenter. This makes a novice or experimental contribution less likely."

You'll notice that this has been a complaint of veteran contributors as
well; WIP patches either get no review, or get reviewed as if they were
expected to be committable.

The person who e-mailed me suggests some form of PostgreSQL Incubator as
a solution.   I'm not sure about that, but it does seem to me that we
need somewhere or some way that people can submit patches, ideas, git
forks, etc., for discussion without that discussion needing to
immediately move to the cleanliness/maintainability/supportable status
of the patch.

I'm concerned though that if these WIP projects don't get to -hackers,
then their creators won't get the feedback they really need.

Thoughts?

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
"Joshua D. Drake"
Date:
On 04/18/2011 06:38 PM, Robert Haas wrote:
> On Mon, Apr 18, 2011 at 3:17 PM, Josh Berkus<josh@agliodbs.com>  wrote:
>> In any case, I think the answer to this is constructive; better
>> documentation and tools to let submitters get their code into good shape
>> in the first place so that we don't have discussions about formatting.
>> That way we waste *neither* the reviewers' nor the submitters' time.
> Well, I'm all in favor of better documentation, but I think the
> biggest thing we need to do is get the word out:
>
> 1. We realize we have been too trigger-happy sometimes.
> 2. But we really want you to participate.
> 3. And we are trying very hard to do better.
> 4. And please tell us if we screw up, so we can keep working on it.

I think Robert has hit the nail on the head. As I mentioned at #PgWest, 
we are a 1000 person dysfunctional family. David Fetter reminded me 
gently (yes really) that as far as 1000 person families go, we're doing 
pretty good. We are one of the last true communities left.

We need to find a way to let people know that we are only gruff, because 
of experience and that although we can be rough we welcome the 
participation and we try really hard. We are engineers (well, I'm 
not...) but most of us are.

JD

-- 
Command Prompt, Inc. - http://www.commandprompt.com/
PostgreSQL Support, Training, Professional Services and Developement
Organizers of the PostgreSQL Conference - http://www.postgresqlconference.org/
@cmdpromptinc - @postgresconf - 509-416-6579



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Alex Hunsaker
Date:
On Mon, Apr 18, 2011 at 19:50, Josh Berkus <josh@agliodbs.com> wrote:
> You'll notice that this has been a complaint of veteran contributors as
> well; WIP patches either get no review, or get reviewed as if they were
> expected to be committable.

I don't see this changing anytime in the future. We have a hard enough
time getting "finished" patches reviewed.

> The person who e-mailed me suggests some form of PostgreSQL Incubator as
> a solution.   I'm not sure about that, but it does seem to me that we
> need somewhere or some way that people can submit patches, ideas, git
> forks, etc., for discussion without that discussion needing to
> immediately move to the cleanliness/maintainability/supportable status
> of the patch.

Reminds me a bit of what linux is doing with the "staging" tree. I
don't see anyway for that to work with postgres (lower the bar for
-contrib?).

You can fork fairly easy with github nowdays. For example the replace
GEQ with SA is on one of those git sites. Does that mean it gets any
attention? *shrug*


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Mon, Apr 18, 2011 at 9:50 PM, Josh Berkus <josh@agliodbs.com> wrote:
> Robert,
>
>> 1. We realize we have been too trigger-happy sometimes.
>> 2. But we really want you to participate.
>> 3. And we are trying very hard to do better.
>> 4. And please tell us if we screw up, so we can keep working on it.
>
> I received a private offlist email from someone who didn't feel
> comfortable bringing up their issues with this list publicly.  Let me
> quote from it, because I think it pins part of the issue:
>
> "I believe this is due to the current postgresql "commitfest" process
> whereby there is no real way to present new ideas or technologies
> without coming to the table with a fully-baked plan and patch. This is
> obvious even in the name "commitfest" since the expectation is that
> every patch presented is considered ready-to-commit by the patch
> presenter. This makes a novice or experimental contribution less likely."
>
> You'll notice that this has been a complaint of veteran contributors as
> well; WIP patches either get no review, or get reviewed as if they were
> expected to be committable.
>
> The person who e-mailed me suggests some form of PostgreSQL Incubator as
> a solution.   I'm not sure about that, but it does seem to me that we
> need somewhere or some way that people can submit patches, ideas, git
> forks, etc., for discussion without that discussion needing to
> immediately move to the cleanliness/maintainability/supportable status
> of the patch.
>
> I'm concerned though that if these WIP projects don't get to -hackers,
> then their creators won't get the feedback they really need.
>
> Thoughts?

I think the quality of review that WIP patches get depends very much
on how specific the submitter is about what they'd like to get out of
the process.  If you submit a patch and say "I have this cool patch
that allows FTL travel, but it's WIP, please review" then you're
basically asking some poor schmuck to reverse engineer what the patch
is doing, and, when they find problems with it, guess which of those
problems were things that you didn't think of and which were things
that you knew about but haven't gotten around to fixing yet because
you're still working on it.  This is a pretty thankless task for the
reviewer, and it's not surprising that it doesn't go well.  However,
if you say "I have this cool patch that allows FTL travel.  It current
plays havoc with the transporter beams and the dilithium crystals tend
to shatter if you exceed Warp 3, but I'd like to get a check as to
whether the basic design is sound, and if anyone can see why the
Heisenburg compensator is destabilizing, please let me know", your
chances of getting some useful feedback are pretty good.  Sometimes it
even provokes a rather competitive spot-the-bug race...

Also, I think the reason why we have a process called CommitFest and
not a process called BrainstormingFest is because, when we didn't have
a CommitFest process, patches fell on the floor.  Since we've added
that process, that problem has largely gone away.  But it is generally
not difficult to get a review of a "big idea" for which no code has
been written yet - in fact it's often much faster and easier than
getting a patch reviewed.  It's true that there have been occasional
times when people have gotten lightly toasted for bringing up big new
ideas in the middle of a CF or beta period, but I think we've gotten
less pedantic about that.  Certainly, there are no shortage of ideas
that have been proposed and commented on over the last few weeks, even
as we have been working to get 9.1beta1 out the door.  Code is not
really getting reviewed right now, but ideas *are*.  I'm not going to
claim that this works perfectly: the way that ideas are presented and
the relative level of interest and/or exhaustion of the people
responding certainly play a role, but it is a pretty rare for an email
to -hackers to get no answer at all.  Maybe we need some formal
process here just to make people more comfortable, but it's not
necessary from a workflow perspective.

Thinking back over the kinds of things that have lead to people
getting jumped on, I think I can identify a pattern: people tend to
get jumped on when they allege that our code sucks, or that they're
smarter than we are.  Whether or not they actually meant to imply
those things turns out not to matter - it rubs people the wrong way,
and everyone's a volunteer, so when you rub them the wrong way, they
get annoyed.  I make an effort, as I think most of us do, to be aware
that just because someone makes an annoying remark doesn't necessarily
mean that they are an annoying person; it just means they haven't
quite figured it all out yet.  But there are still people who get
flamed that way far more than they probably deserve.  That's an area
we can improve, but in the meantime, approaching the topic with a bit
of humility goes a long way.  I can't remember the last time someone
said "I was thinking about working on ... and I thought I might
approach it by ... Does this seem like a good idea?  Is it likely to
be too hard for me to tackle?  My skillset is ..." and got flamed for
it.  Some people here (myself included) get a bit pricklier than we
probably ought to from time to time, but everyone is well-meaning and
sincerely wants to help.  The list of users who have had Tom fix a bug
for them within hours of posting a question is not short, and the list
of people who have spent time and energy helping newcomers get started
with PostgreSQL tuning, hacking, or whatever is very long.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Christopher Browne
Date:
On Mon, Apr 18, 2011 at 11:15 PM, Alex Hunsaker <badalex@gmail.com> wrote:
> On Mon, Apr 18, 2011 at 19:50, Josh Berkus <josh@agliodbs.com> wrote:
>> You'll notice that this has been a complaint of veteran contributors as
>> well; WIP patches either get no review, or get reviewed as if they were
>> expected to be committable.
>
> I don't see this changing anytime in the future. We have a hard enough
> time getting "finished" patches reviewed.

Sadly so.

As much as I think we have gotten a LOT of useful milage out of the
"commitfest" concept, it does, conceptually, have a strong bias
(including in its very name) towards the assumption that changes are
pretty much ready to commit.

Two items still undergoing work (collations, sync rep) weren't at that
level of readiness, needing some mere "dusting off" to make them
ready.  Rather, they needed substantial examination and modification
before they'd be ready.  And, while this has doubtless aroused some
ire, it doesn't intrinsically make those items "broken."

The Apache guys may be onto something in having the "incubator"
moniker, for things that aren't "so ready we're calling them
Commitable."

There may be merit to separating out "easy to commit" and "tougher to
commit" items, and having different kinds of pickiness for them, the
former being good fodder for "Easy CommitFest" and the latter being
"PG Incubation."

Though I'm not sure the latter makes it any easier to get tough
features like synchronous replication into place.

>> The person who e-mailed me suggests some form of PostgreSQL Incubator as
>> a solution.   I'm not sure about that, but it does seem to me that we
>> need somewhere or some way that people can submit patches, ideas, git
>> forks, etc., for discussion without that discussion needing to
>> immediately move to the cleanliness/maintainability/supportable status
>> of the patch.
>
> Reminds me a bit of what linux is doing with the "staging" tree. I
> don't see anyway for that to work with postgres (lower the bar for
> -contrib?).
>
> You can fork fairly easy with github nowdays. For example the replace
> GEQ with SA is on one of those git sites. Does that mean it gets any
> attention? *shrug*

Well, the project hasn't been on Git for all that spectacularly long a
time, so the comfort level with managing via forks maybe isn't quite
there yet.

Forking isn't as magically delicious as GitHub might make some
imagine; it's fine and useful to have a bunch of forks, and eventually
merge useful ones, when they are remaining pretty close together, and
don't conflict.  That's likely to work out happily for features that
are essentially independent.  If you and I are hacking on different
contrib modules, that's pretty "essentially independent."

Unfortunately, deeper features are more likely to be more
interdependent, and forks aren't so readily productive in that case.

If we hack around with formatting, that would muck with *everything*
else, as an even worse "for instance."
--
When confronted by a difficult problem, solve it by reducing it to the
question, "How would the Lone Ranger handle this?"


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Mon, Apr 18, 2011 at 11:50 PM, Christopher Browne <cbbrowne@gmail.com> wrote:
> Two items still undergoing work (collations, sync rep) weren't at that
> level of readiness, needing some mere "dusting off" to make them
> ready.  Rather, they needed substantial examination and modification
> before they'd be ready.  And, while this has doubtless aroused some
> ire, it doesn't intrinsically make those items "broken."

I don't think it really aroused that much ire.  It's pretty clear that
both of those patches cost us something on the schedule, and I would
have preferred to see them committed sooner and with fewer bugs.  But
they are great features.  Unfortunately, we have a tendency to leave
things to the last minute, and that's something I think we could
improve.  We have gotten a bit better but there is clearly room for
further improvement.  With beta having gotten pushed out to the end of
the month, there is a real chance that we are going to end up
releasing in the fall again, and I would have much preferred July 1.
But given how long CF4 lasted and how much surgery was required
afterwards, it was an unfixable problem.  It's not going to get any
better unless we get more serious about getting these big features
done early in the cycle, or postponing them to the next release if
they aren't.  Anyway, I'm drifting off topic: nothing against the
patches, at least on my part, just want to make the schedule.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Josh Berkus <josh@agliodbs.com> writes:
> I received a private offlist email from someone who didn't feel
> comfortable bringing up their issues with this list publicly.  Let me
> quote from it, because I think it pins part of the issue:

> "I believe this is due to the current postgresql "commitfest" process
> whereby there is no real way to present new ideas or technologies
> without coming to the table with a fully-baked plan and patch. This is
> obvious even in the name "commitfest" since the expectation is that
> every patch presented is considered ready-to-commit by the patch
> presenter. This makes a novice or experimental contribution less likely."

As Robert noted, the purpose of the commitfest mechanism is mostly to
ensure that patches that *are* committable, or close to it, don't fall
through the cracks.  I'm not sure we're doing anybody any favors by
trying to shoehorn reviews of WIP ideas into that same process.  At the
very least it seems we'd need a different set of review guidelines for
WIP items, and we don't have one.

I think useful reviewing of WIP stuff has to focus much more on design
concepts and much less on code reading.  The reason why the mmap patch
was getting such negative feedback was that there was no way to provide
such a review except by reverse-engineering the design out of some very
messily-presented code.  So if we're going to do anything about this,
what we have to do is tell people that the first thing to present for
a WIP review is a design document.  If they feel a need to write some
throwaway code to help them clarify their ideas, fine ... but *don't
show us that code*.  Write a design document.  Get that reviewed.
Then see about coding it, or bringing your first-draft code up to the
point where it can stand the light of day.

I don't know if we need a formal process akin to CFs for reviewing
design documents.  I think people are usually plenty willing to discuss
ideas on -hackers, unless maybe you hit them at a particularly bad time
like when they're already burnt out towards the end of a CF.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Alvaro Herrera
Date:
Excerpts from Tom Lane's message of mar abr 19 03:34:34 -0300 2011:

> As Robert noted, the purpose of the commitfest mechanism is mostly to
> ensure that patches that *are* committable, or close to it, don't fall
> through the cracks.  I'm not sure we're doing anybody any favors by
> trying to shoehorn reviews of WIP ideas into that same process.  At the
> very least it seems we'd need a different set of review guidelines for
> WIP items, and we don't have one.

I think this is historical revisionism.  Commitfests were mostly created
because of pressure due to the lateness of the HOT patch.  Probably
there were other factors too but this is likely the single most
important reason.  (I think the term "commitfest" was coined later, but
I don't think this invalidates my point.)

And the way we considered things at the time is that we had failed to
timely review the concepts in the WIP HOT patch that was presented.  So
we wanted to ensure that we provided good feedback to WIP patches (to
all patches really) to avoid this failure from repeating.  All patches
*and WIP ideas* were supposed to be reviewed by someone, and if they
were to be rejected, some rationale was to be provided.

Somewhere down the line this seems to have been forgotten and we are now
using commitfests just to track finished patches.

So if we want to stick to the original principles we should have some
sort of "different set of review guidelines".  Or perhaps we could just
decide that we don't care much about this problem and toss it aside.

Maybe this is something to discuss at the next developer's meeting.

-- 
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Alvaro Herrera <alvherre@commandprompt.com> writes:
> I think this is historical revisionism. ...
> Somewhere down the line this seems to have been forgotten and we are now
> using commitfests just to track finished patches.

> So if we want to stick to the original principles we should have some
> sort of "different set of review guidelines".  Or perhaps we could just
> decide that we don't care much about this problem and toss it aside.

Well, I absolutely think that we need to encourage people to get
feedback at the design and prototype stages.  The problem with the
commitfest mechanism for that is that when you are trying to work out a
patch, you don't want to wait around for a couple months for comments.
The time delay that's built into the CF process means that it's
fundamentally not very good for anything except finished patches that
can sit on a shelf for awhile before they get applied.

I think that ideally, WIP reviews would be something that happens
quickly on pgsql-hackers, and probably it would be best if they were
explicitly *not* encouraged while a CF is on.  I know that I tend to see
discussions of unfinished patches as something of a distraction when
I'm up to my ears in committing finished ones, and certainly there's
less mental bandwidth available then.

> Maybe this is something to discuss at the next developer's meeting.

I'd rather talk about it on-list so we can get comments from a wider
circle of people.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Greg Stark
Date:
On Wed, Apr 20, 2011 at 5:21 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Well, I absolutely think that we need to encourage people to get
> feedback at the design and prototype stages.  The problem with the
> commitfest mechanism for that is that when you are trying to work out a
> patch, you don't want to wait around for a couple months for comments.
> The time delay that's built into the CF process means that it's
> fundamentally not very good for anything except finished patches that
> can sit on a shelf for awhile before they get applied.

From my point of view I definitely thought the rationale for
commitfests was as a kind of checkpoint to make sure there weren't any
developers waiting for feedback for a long time. My concurrent index
build patch ended up needing to be reworked and I would have liked to
be involved but it wasn't until feature freeze that you found all
these problems and then it was too late to wait for me to recode
things instead of having you just do it.

I admit though this whole concept of "finished patches" seems foreign
to me. I always have additional stuff I want to do and if the patch
sits on the shelf I'm essentially stuck unable to work on the next
great thing that that patch enables. Developers either have the option
to go off on their own with no feedback and risk having initial
assumptions questioned later and all their work invalidated or go and
work on something unrelated leaving this direction stunted with only
one round of features implemented. I think this is how we ended up
with partitioning that's only halfway useful and selinux that had tons
of code written that needed to be reworked.

Core developers attention is precious and we can't really dictate that
Tom must respond to every email within a week or anything crazy like
that. The commitfests are a dramatic improvement over waiting until
feature freeze which was what was happening before. They also help
bring in new committers and having Robert and Heikki and Peter and
others giving substantive feedback has also improved things
dramatically.

To use a database analogy I think of the commitfests as a checkpoint
-- that doesn't mean we don't also need bgwriter and don't
occasionally need to flush dirty buffers to enable the database to
make progress in the mean-time. But if we didn't have checkpoints at
all things would definitely fall through the cracks and get lost to
bitrot and brainfade.

--
greg


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Wed, Apr 20, 2011 at 12:21 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Alvaro Herrera <alvherre@commandprompt.com> writes:
>> I think this is historical revisionism. ...
>> Somewhere down the line this seems to have been forgotten and we are now
>> using commitfests just to track finished patches.
>
>> So if we want to stick to the original principles we should have some
>> sort of "different set of review guidelines".  Or perhaps we could just
>> decide that we don't care much about this problem and toss it aside.
>
> Well, I absolutely think that we need to encourage people to get
> feedback at the design and prototype stages.  The problem with the
> commitfest mechanism for that is that when you are trying to work out a
> patch, you don't want to wait around for a couple months for comments.
> The time delay that's built into the CF process means that it's
> fundamentally not very good for anything except finished patches that
> can sit on a shelf for awhile before they get applied.
>
> I think that ideally, WIP reviews would be something that happens
> quickly on pgsql-hackers, and probably it would be best if they were
> explicitly *not* encouraged while a CF is on.  I know that I tend to see
> discussions of unfinished patches as something of a distraction when
> I'm up to my ears in committing finished ones, and certainly there's
> less mental bandwidth available then.

Ditto.

Unfortunately, my memory of this project only goes back to about
September 2008, which isn't far enough to remember why CommitFests
were created in the first place.  So Alvaro may be correct in saying
that things have mutated over time, but that isn't necessarily a bad
thing.  Maybe we've settled into something that works reasonably well.Or maybe we should make some changes; nothing is
setin stone. 

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Peter Eisentraut
Date:
On Wed, 2011-04-20 at 17:52 +0100, Greg Stark wrote:
> I admit though this whole concept of "finished patches" seems foreign
> to me. I always have additional stuff I want to do and if the patch
> sits on the shelf I'm essentially stuck unable to work on the next
> great thing that that patch enables. Developers either have the option
> to go off on their own with no feedback and risk having initial
> assumptions questioned later and all their work invalidated or go and
> work on something unrelated leaving this direction stunted with only
> one round of features implemented. I think this is how we ended up
> with partitioning that's only halfway useful and selinux that had tons
> of code written that needed to be reworked.

Yeah, there appear to be occasional assumptions that one ought to work
on one major feature per release, and ideally you'd have the plan ready
for the first commit fest and the code mostly ready for the third commit
fest.  Whereas I agree with you that it's often rather the case that you
want to work on say three incremental features, and order for that to
work out under this process, you really have to get the first increment
perfect for the first commit fest already.  Which is difficult if no one
pays attention until the commit fest starts.



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Peter Eisentraut
Date:
On Wed, 2011-04-20 at 12:21 -0400, Tom Lane wrote:
> Well, I absolutely think that we need to encourage people to get
> feedback at the design and prototype stages.  The problem with the
> commitfest mechanism for that is that when you are trying to work out
> a patch, you don't want to wait around for a couple months for
> comments.  The time delay that's built into the CF process means that
> it's fundamentally not very good for anything except finished patches
> that can sit on a shelf for awhile before they get applied.
> 
> I think that ideally, WIP reviews would be something that happens
> quickly on pgsql-hackers, and probably it would be best if they were
> explicitly *not* encouraged while a CF is on.  I know that I tend to
> see discussions of unfinished patches as something of a distraction
> when I'm up to my ears in committing finished ones, and certainly
> there's less mental bandwidth available then.

We'll the current process certainly places a lot of emphasis on the
"finishing" part.  You have commit fests that nominally account for 50%
of development time, and then beta, RC, limbo, backbranch releases -- I
blogged about this a while ago, if you follow all these guidelines and
encouragements, you are left with all of about 20 days per year for
discussion, collaborative planning and coding.  Which is obviously
silly, which is why the process breaks down.  People do other things as
commit fests fade out, but they subconsciously fear they will get the
stink for it, so public discussion and planning is effectively stifled.

I think we should put less temporal emphasis on the finishing part, but
use the time better.  I would imagine one commit fest per month, but
it's only a week long.  Then everyone can really concentrate on the
commit fest, people get faster feedback, but there is ultimately more
time to do other things.  Something to think about.



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Josh Berkus
Date:
Robert,

> Unfortunately, my memory of this project only goes back to about
> September 2008, which isn't far enough to remember why CommitFests
> were created in the first place.  So Alvaro may be correct in saying
> that things have mutated over time, but that isn't necessarily a bad
> thing.  Maybe we've settled into something that works reasonably well.
>  Or maybe we should make some changes; nothing is set in stone.

Review of design concepts and WIP patches has *always* been a problem
for this project.  Andrew Sullivan bitched about it at some length back
in 2004 ("Why there is no traffic on pgsql-replicationhooks", but
Andrew's blog is down now unfortunately).  And I've gotten complaints
from numerous people: the Drizzle student, the person who e-mailed me,
Afilias, Greenplum, Aster Data, others.  It's just a broken process, and
it particularly leads PostgreSQL forks to not contribute back stuff.

We tell people to submit a design concept, but then such submissions are
often ignored.  When they're not ignored, they often are subject to
either extreme bikeshedding or a lot of negativity around things the
author hasn't implemented yet ... even if the author warns that they're
not implemented.

(btw, I'm not talking about the MMAP patch here, which has gotten
excellent review at this point.  I'm talking about a lot of other patches)

I think that Robert is right and what we need is a completely different
process for WIP patches and design concepts.  It's pretty clear that
none of the processes we've tried so far ("just post it to
pgsql-hackers", "get a submission mentor" and "commitfest") have worked
consistently.

So in the spirit of NOT reinventing the wheel: ReviewBoard.  Yes,
really.  One of the big issues with working through design reviews etc.
on this mailing list is the lack of continuity and timeliness in
comments on the idea/WIP patch.  Having an interface which presents all
of the discussion around a specific  patch in a threaded and
chronological way would help cut down on bikeshedding and dogpiling, as
well as allowing both the idea/patch author to review all commentary in
a coherent way.

Maybe we don't want to use ReviewBoard specifically.   Maybe we want to
use bugzilla or Crucible or Redmine something more specific for
patch/spec review.  But I think it's time to try something else, maybe
several other things.

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Peter Eisentraut <peter_e@gmx.net> writes:
> I think we should put less temporal emphasis on the finishing part, but
> use the time better.  I would imagine one commit fest per month, but
> it's only a week long.  Then everyone can really concentrate on the
> commit fest, people get faster feedback, but there is ultimately more
> time to do other things.  Something to think about.

Yeah, maybe.  To do that, we'd have to strongly resist the temptation to
spend a lot of time fixing up submitted patches --- if it's not pretty
darn close to committable, back it goes.  But that might be a good thing
all around.  I find this idea attractive.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andres Freund
Date:
On Wednesday, April 20, 2011 08:50:04 PM Tom Lane wrote:
> > I think we should put less temporal emphasis on the finishing part, but
> > use the time better.  I would imagine one commit fest per month, but
> > it's only a week long.  Then everyone can really concentrate on the
> > commit fest, people get faster feedback, but there is ultimately more
> > time to do other things.  Something to think about.
> Yeah, maybe.  To do that, we'd have to strongly resist the temptation to
> spend a lot of time fixing up submitted patches --- if it's not pretty
> darn close to committable, back it goes.  But that might be a good thing
> all around.  I find this idea attractive.
Actually as a patch submitter I would somewhat prefer that as well. Its not 
exactly easy to learn what wasn't optimal with your patch at times.

On the other hand for some issues its pretty hard to fix the more involved 
issues without e.g. Tom's involvement.

Andres


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Wed, Apr 20, 2011 at 2:39 PM, Josh Berkus <josh@agliodbs.com> wrote:
> Review of design concepts and WIP patches has *always* been a problem
> for this project.  Andrew Sullivan bitched about it at some length back
> in 2004 ("Why there is no traffic on pgsql-replicationhooks", but
> Andrew's blog is down now unfortunately).  And I've gotten complaints
> from numerous people: the Drizzle student, the person who e-mailed me,
> Afilias, Greenplum, Aster Data, others.  It's just a broken process, and
> it particularly leads PostgreSQL forks to not contribute back stuff.
>
> We tell people to submit a design concept, but then such submissions are
> often ignored.

Please provide the evidence that this is a problem that exists now, as
opposed to seven years ago.  I leave pgsql-hackers emails marked
unread until they have gotten a response, especially if it's something
important like a design proposal.  I have 10 unread threads at the
moment; and I don't think any of them are design proposals except
possibly "Still more REINDEX fun", which was posted 9 minutes ago by
Tom - presumably not the case you are concerned about.  I have worked
extremely hard to make sure that we do NOT ignore such submissions,
and I would like to hold your feet to the fire on this one a little
bit: let's hear the list of design ideas that have been proposed in
the last year and been ignored.  If the process is as bad as you are
alleging, you should find it easy to come up with numerous, recent
examples.  I bet you can't.

> When they're not ignored, they often are subject to
> either extreme bikeshedding or a lot of negativity around things the
> author hasn't implemented yet ... even if the author warns that they're
> not implemented.

I concede that this happens, but I don't believe it happens nearly as
often as it used to, and, again, let's have some recent examples.  I
don't care what happened three years ago; a lot has changed in the
last three years.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Josh Berkus
Date:
On 4/20/11 12:00 PM, Robert Haas wrote:
> Please provide the evidence that this is a problem that exists now, as
> opposed to seven years ago.

Since you're clearly already made up your mind that no problem exists, I
don't have the energy to fight it out with you.

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Wed, Apr 20, 2011 at 2:53 PM, Andres Freund <andres@anarazel.de> wrote:
> On Wednesday, April 20, 2011 08:50:04 PM Tom Lane wrote:
>> > I think we should put less temporal emphasis on the finishing part, but
>> > use the time better.  I would imagine one commit fest per month, but
>> > it's only a week long.  Then everyone can really concentrate on the
>> > commit fest, people get faster feedback, but there is ultimately more
>> > time to do other things.  Something to think about.
>> Yeah, maybe.  To do that, we'd have to strongly resist the temptation to
>> spend a lot of time fixing up submitted patches --- if it's not pretty
>> darn close to committable, back it goes.  But that might be a good thing
>> all around.  I find this idea attractive.
> Actually as a patch submitter I would somewhat prefer that as well. Its not
> exactly easy to learn what wasn't optimal with your patch at times.
>
> On the other hand for some issues its pretty hard to fix the more involved
> issues without e.g. Tom's involvement.

This would amount to reducing the amount of time we spend
in-CommitFest from 50% to slightly less than 25%.  That would
certainly be pleasant from my point of view, but for the average patch
to get the same amount of attention, we'd need twice as many
volunteers, or the existing people to volunteer twice as much time, or
everyone to work twice as fast as they already are.  That's not
impossible, if the new system inspires more people to contribute, but
2x is a lot, especially when you correct for relative skill levels:
we're not going to find another Tom Lane.

Still, it's an interesting thought.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andres Freund
Date:
On Wednesday, April 20, 2011 08:39:47 PM Josh Berkus wrote:
> Robert,
> 
> > Unfortunately, my memory of this project only goes back to about
> > September 2008, which isn't far enough to remember why CommitFests
> > were created in the first place.  So Alvaro may be correct in saying
> > that things have mutated over time, but that isn't necessarily a bad
> > thing.  Maybe we've settled into something that works reasonably well.
> > 
> >  Or maybe we should make some changes; nothing is set in stone.
> 
> Review of design concepts and WIP patches has *always* been a problem
> for this project.  Andrew Sullivan bitched about it at some length back
> in 2004 ("Why there is no traffic on pgsql-replicationhooks", but
> Andrew's blog is down now unfortunately).  And I've gotten complaints
> from numerous people: the Drizzle student, the person who e-mailed me,
> Afilias, Greenplum, Aster Data, others.  It's just a broken process, and
> it particularly leads PostgreSQL forks to not contribute back stuff.
Well. But very few company people to contribute back in reviewing stuff from 
others. At least in the time I have somewhat regularly 

> We tell people to submit a design concept, but then such submissions are
> often ignored.  When they're not ignored, they often are subject to
> either extreme bikeshedding or a lot of negativity around things the
> author hasn't implemented yet ... even if the author warns that they're
> not implemented.
I can see that point.

> I think that Robert is right and what we need is a completely different
> process for WIP patches and design concepts.  It's pretty clear that
> none of the processes we've tried so far ("just post it to
> pgsql-hackers", "get a submission mentor" and "commitfest") have worked
> consistently.
> 
> So in the spirit of NOT reinventing the wheel: ReviewBoard.  Yes,
> really.  One of the big issues with working through design reviews etc.
> on this mailing list is the lack of continuity and timeliness in
> comments on the idea/WIP patch.  Having an interface which presents all
> of the discussion around a specific  patch in a threaded and
> chronological way would help cut down on bikeshedding and dogpiling, as
> well as allowing both the idea/patch author to review all commentary in
> a coherent way.
I don't believe a second that problem is solved by any tool. In my opinion 
there simply are very few people being able to do in-depth reviews of complex 
patches. And those are also needed to implement complex features or do parts 
of features others could not do.

A RRR like process doesn't really help in those cases except catch the most 
obvious problems.


Andres


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
"Joshua D. Drake"
Date:
On 04/20/2011 12:05 PM, Josh Berkus wrote:
> On 4/20/11 12:00 PM, Robert Haas wrote:
>> Please provide the evidence that this is a problem that exists now, as
>> opposed to seven years ago.
> Since you're clearly already made up your mind that no problem exists, I
> don't have the energy to fight it out with you.
Well, you aren't fighting alone. We have significant problems in this 
area. As you said, we always have. There is also a bizarre, almost 
insane objection to using tools that "aren't invented here" to solve 
problems. The problems you (Josh) present are real, regardless of 
Robert's opinion. The thing that is important for everyone to remember 
is PERCEPTION IS REALITY.

If people PERCEIVE there is a problem, THERE IS A PROBLEM.

So Robert, with respect to your "show me the money", the money is at 
your feet on the floor. JB and I can list multitudes of hackers and 
contributors who have the perception of this problem and that perception 
is hurting the project because frankly, Astor Data isn't going to waste 
it's valuable time (money) fighting our community. We have to make it 
damn freaking easy for them or we lose their interest, and thus the 
community loses.
From the whales of discontentment society,

JD



-- 
Command Prompt, Inc. - http://www.commandprompt.com/
PostgreSQL Support, Training, Professional Services and Developement
Organizers of the PostgreSQL Conference - http://www.postgresqlconference.org/
@cmdpromptinc - @postgresconf - 509-416-6579



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andres Freund
Date:
On Wednesday, April 20, 2011 09:09:48 PM Robert Haas wrote:
> On Wed, Apr 20, 2011 at 2:53 PM, Andres Freund <andres@anarazel.de> wrote:
> > On Wednesday, April 20, 2011 08:50:04 PM Tom Lane wrote:
> >> Yeah, maybe.  To do that, we'd have to strongly resist the temptation to
> >> spend a lot of time fixing up submitted patches --- if it's not pretty
> >> darn close to committable, back it goes.  But that might be a good thing
> >> all around.  I find this idea attractive.
> > Actually as a patch submitter I would somewhat prefer that as well. Its
> > not exactly easy to learn what wasn't optimal with your patch at times.
> > On the other hand for some issues its pretty hard to fix the more
> > involved issues without e.g. Tom's involvement.
> This would amount to reducing the amount of time we spend
> in-CommitFest from 50% to slightly less than 25%.  That would
> certainly be pleasant from my point of view, but for the average patch
> to get the same amount of attention, we'd need twice as many
> volunteers, or the existing people to volunteer twice as much time, or
> everyone to work twice as fast as they already are.  That's not
> impossible, if the new system inspires more people to contribute, but
> 2x is a lot, especially when you correct for relative skill levels:
> we're not going to find another Tom Lane.
> Still, it's an interesting thought.
Additional points:
* perhaps it also frees up time if committers balk earlier if a patch doesn't 
meet some requirement
* Patch submitters learn more:   * so they submit better patches in the future   * so they can apply the same standards
whenthey review other patches
 

Andres


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Wed, Apr 20, 2011 at 3:13 PM, Joshua D. Drake <jd@commandprompt.com> wrote:
> On 04/20/2011 12:05 PM, Josh Berkus wrote:
>>
>> On 4/20/11 12:00 PM, Robert Haas wrote:
>>>
>>> Please provide the evidence that this is a problem that exists now, as
>>> opposed to seven years ago.
>>
>> Since you're clearly already made up your mind that no problem exists, I
>> don't have the energy to fight it out with you.
>
> Well, you aren't fighting alone. We have significant problems in this area.
> As you said, we always have. There is also a bizarre, almost insane
> objection to using tools that "aren't invented here" to solve problems. The
> problems you (Josh) present are real, regardless of Robert's opinion. The
> thing that is important for everyone to remember is PERCEPTION IS REALITY.
>
> If people PERCEIVE there is a problem, THERE IS A PROBLEM.

Absolutely.  And I am perfectly well aware that we have screwed this
up from time to time.  But I also know that I have spent a very large
amount of time over the last few years trying to improve things.  It
would be nice to know whether that has had any impact.  If it hasn't,
then half of what I have spent the last two years doing has been a
waste of time.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andres Freund
Date:
On Wednesday, April 20, 2011 08:53:34 PM Andres Freund wrote:
> On Wednesday, April 20, 2011 08:50:04 PM Tom Lane wrote:
> > > I think we should put less temporal emphasis on the finishing part, but
> > > use the time better.  I would imagine one commit fest per month, but
> > > it's only a week long.  Then everyone can really concentrate on the
> > > commit fest, people get faster feedback, but there is ultimately more
> > > time to do other things.  Something to think about.
> > 
> > Yeah, maybe.  To do that, we'd have to strongly resist the temptation to
> > spend a lot of time fixing up submitted patches --- if it's not pretty
> > darn close to committable, back it goes.  But that might be a good thing
> > all around.  I find this idea attractive.
> 
> Actually as a patch submitter I would somewhat prefer that as well. Its not
> exactly easy to learn what wasn't optimal with your patch at times.
Perhaps we should adapt something like the kernel's checkpatch.pl for our 
needs?
I.e. something that checks that the most obvious style issues are addressed 
(tabs, trailing whitespaces, spacing around braces etc).

Andres


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Robert Haas <robertmhaas@gmail.com> writes:
> On Wed, Apr 20, 2011 at 2:53 PM, Andres Freund <andres@anarazel.de> wrote:
>> On Wednesday, April 20, 2011 08:50:04 PM Tom Lane wrote:
>>> Yeah, maybe.  To do that, we'd have to strongly resist the temptation to
>>> spend a lot of time fixing up submitted patches --- if it's not pretty
>>> darn close to committable, back it goes.  But that might be a good thing
>>> all around.  I find this idea attractive.

>> Actually as a patch submitter I would somewhat prefer that as well. Its not
>> exactly easy to learn what wasn't optimal with your patch at times.
>> On the other hand for some issues its pretty hard to fix the more involved
>> issues without e.g. Tom's involvement.

> This would amount to reducing the amount of time we spend
> in-CommitFest from 50% to slightly less than 25%.  That would
> certainly be pleasant from my point of view, but for the average patch
> to get the same amount of attention, we'd need twice as many
> volunteers, or the existing people to volunteer twice as much time, or
> everyone to work twice as fast as they already are.

Well, no, that's not the whole story.  To me, what the above idea
implies is shifting more of the burden of fixing up patches away from
the committer and back to the patch author.  Instead of spending time
fixing up not-quite-ready patches myself, I'd be much more ready to
tell the patch author "do X, Y, and Z, and come back next month".

From the committers' standpoint, this is a great idea precisely because
it suggests we might get to put only 25% and not 50% of our time into
commitfests.  But it also makes the work more distributed, and it forces
patch authors to learn the things that committers might otherwise have
done for them silently, which in the long run will make everything work
better.

The key point is that we do have to have much more frequent commitfests.
It's hard to bounce back a patch when you know it will then be delayed
two months, especially if the patch is already two months old and the
author has probably forgotten half of it himself.  For me anyway,
"I'll just take half a day and make this look the way I think it should"
is a continual temptation.  A shorter CF cycle would weaken the argument
to do that.

I haven't spent any time in the role of a non-committer reviewer, but
I think that the same dynamic might work for reviewers.  Basically
what a short cycle would do is encourage people to hit the high points
and turn the review around quickly, dumping the big issues back into the
patch author's lap for fixing.  You wouldn't spend time sweating details
until the patch had gotten into a state that justified it.  Of course
we'd need to tweak the review guidelines to encourage this sort of
multiple-iterations review approach --- right now the guidelines are
pretty much one-size-fits-all, and this type of approach cannot work
with that.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Josh Berkus
Date:
On 4/20/11 12:35 PM, Tom Lane wrote:
> Well, no, that's not the whole story.  To me, what the above idea
> implies is shifting more of the burden of fixing up patches away from
> the committer and back to the patch author.  Instead of spending time
> fixing up not-quite-ready patches myself, I'd be much more ready to
> tell the patch author "do X, Y, and Z, and come back next month".

Yes, definitely!  For that matter, booting a patch which got no review
is less of a problem if we're only booting it for 3 weeks.

The whole purpose of the CFs was not to help submitters -- it was to
help reviewers.   If we just wanted to help submitters, we'd do
Continuous Integration, and review all the time.  But the reviewers need
"time off".

I think we should try this for 9.2.  Given the accumulation between then
and now, I think the first CF should be 2 weeks, and then we can move to
monthly/weeklong CFs after that.  So it would look like:

CF1: July 16-31
CF2: August 1-7
CF3: September 1-7
CF4: October 1-7
CF5: November 1-7
CF6: December 1-7
CF7: January 3-10
CF8: February until done

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Josh Berkus
Date:
Robert,

> Absolutely.  And I am perfectly well aware that we have screwed this
> up from time to time.  But I also know that I have spent a very large
> amount of time over the last few years trying to improve things.  It
> would be nice to know whether that has had any impact.  If it hasn't,
> then half of what I have spent the last two years doing has been a
> waste of time.

That would take pretty significant research; it's not like we have a
database of idea/WIP submissions.  It's all e-mail.

Not that it wouldn't be worth doing, but it would be an entire day of
someone's time.

BTW, I do still believe that step 1 is tremendously expanding the "so
you want to submit a patch" documentation, and linking it in many places
so that newbies read it.  On my TODO list.

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Wed, Apr 20, 2011 at 3:05 PM, Josh Berkus <josh@agliodbs.com> wrote:
> On 4/20/11 12:00 PM, Robert Haas wrote:
>> Please provide the evidence that this is a problem that exists now, as
>> opposed to seven years ago.
>
> Since you're clearly already made up your mind that no problem exists, I
> don't have the energy to fight it out with you.

It is not possible for me to work any harder on anything than I have
worked on this problem.  I do not deny the existence of the problem.
But I believe that we have greatly mitigated it in the last few
release cycles, and that much of what remains is a problem of
perception, not reality.

You can disagree, but if no one has the energy to find real examples
and talk about them, then it is hard to see how we will be able to
improve the situation.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Peter Eisentraut <peter_e@gmx.net> writes:
> I would imagine one commit fest per month, but
> it's only a week long.

BTW, just as a thought experiment: what about a one-day CF once a week?
"Patch Tuesdays", if you will.  Spend all day reviewing/committing,
bounce back whatever is not ready, patch authors try again next week.

Really large patches are not going to fit into that paradigm, probably,
but an awful lot of stuff would --- and it might help encourage more
incremental development of the big ones, too.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Magnus Hagander
Date:
On Wed, Apr 20, 2011 at 21:54, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Peter Eisentraut <peter_e@gmx.net> writes:
>> I would imagine one commit fest per month, but
>> it's only a week long.
>
> BTW, just as a thought experiment: what about a one-day CF once a week?
> "Patch Tuesdays", if you will.  Spend all day reviewing/committing,
> bounce back whatever is not ready, patch authors try again next week.

I think that would pretty much kill the process for any committer who
is not employed to work full-time on postgresql *development*. Those
who have other dayjobs (which may well be postgresql consulting or
training or whatever) will probably end up dealing with significantly
fewer patches, leaving even more of the burden on those who do have
the dedicated schedule. I know I don't do as much reviewing/comitting
as I'd like to do during the commitfests, but with a process like
that, it would probably become more or less zero.

--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andrew Dunstan
Date:

On 04/20/2011 04:09 PM, Magnus Hagander wrote:
> On Wed, Apr 20, 2011 at 21:54, Tom Lane<tgl@sss.pgh.pa.us>  wrote:
>> Peter Eisentraut<peter_e@gmx.net>  writes:
>>> I would imagine one commit fest per month, but
>>> it's only a week long.
>> BTW, just as a thought experiment: what about a one-day CF once a week?
>> "Patch Tuesdays", if you will.  Spend all day reviewing/committing,
>> bounce back whatever is not ready, patch authors try again next week.
> I think that would pretty much kill the process for any committer who
> is not employed to work full-time on postgresql *development*. Those
> who have other dayjobs (which may well be postgresql consulting or
> training or whatever) will probably end up dealing with significantly
> fewer patches, leaving even more of the burden on those who do have
> the dedicated schedule. I know I don't do as much reviewing/comitting
> as I'd like to do during the commitfests, but with a process like
> that, it would probably become more or less zero.
>

Yeah, I can't organize my time that way either.

cheers

andrew


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Andrew Dunstan <andrew@dunslane.net> writes:
> On 04/20/2011 04:09 PM, Magnus Hagander wrote:
>> On Wed, Apr 20, 2011 at 21:54, Tom Lane<tgl@sss.pgh.pa.us>  wrote:
>>> BTW, just as a thought experiment: what about a one-day CF once a week?
>>> "Patch Tuesdays", if you will.  Spend all day reviewing/committing,
>>> bounce back whatever is not ready, patch authors try again next week.

>> I think that would pretty much kill the process for any committer who
>> is not employed to work full-time on postgresql *development*.

> Yeah, I can't organize my time that way either.

True, and any fixed day of the week would let out X number of people
anyway.  But ignoring scheduling difficulties, my point here is that
it seems like the shorter the cycle, the better, for a lot of purposes.
Can we do any better than once-a-month, or is that the limit given that
people need flexible schedules within the fest?
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Peter Eisentraut
Date:
On Wed, 2011-04-20 at 11:39 -0700, Josh Berkus wrote:
> Maybe we don't want to use ReviewBoard specifically.   Maybe we want
> to use bugzilla or Crucible or Redmine something more specific for
> patch/spec review.  But I think it's time to try something else, maybe
> several other things.

I had suggested ideatorrent before.  But I agree with you in principle.



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Josh Berkus
Date:
Tom,

> True, and any fixed day of the week would let out X number of people
> anyway.  But ignoring scheduling difficulties, my point here is that
> it seems like the shorter the cycle, the better, for a lot of purposes.
> Can we do any better than once-a-month, or is that the limit given that
> people need flexible schedules within the fest?

Also consider that the PostgreSQL development world represents a lot of
different time zones.  For me to have some dialog with Tatsuo about a
patch, for example, takes at least 24 hours for a simple back-and-forth.

If we were a full-time development shop in a single time zone, we could
use scrum and do a *daily* integration.  Many of my clients do.  But for
a high-distributed volunteer-based organization, I don't think it's
practical.

I also find the one-day-a-week attractive.  It would make patch review
much more immediate.  However, not only would it raise issues with
people's schedules, it would also require us to adopt new tools or
modify the CF code.

Is there anything between one-week-a-month and one-day-a-week?

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Peter Eisentraut
Date:
On Wed, 2011-04-20 at 15:09 -0400, Robert Haas wrote:
> This would amount to reducing the amount of time we spend
> in-CommitFest from 50% to slightly less than 25%.  That would
> certainly be pleasant from my point of view, but for the average patch
> to get the same amount of attention, we'd need twice as many
> volunteers, or the existing people to volunteer twice as much time, or
> everyone to work twice as fast as they already are.

I think in reality people don't spend more than 50% of their time during
commit fests on the commit fest.  By making the commit fests shorter and
tighter, we could perhaps increase that number.  More "quality time" if
you will.



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
"Joshua D. Drake"
Date:
On 04/20/2011 12:22 PM, Robert Haas wrote:
>
>> Well, you aren't fighting alone. We have significant problems in this area.
>> As you said, we always have. There is also a bizarre, almost insane
>> objection to using tools that "aren't invented here" to solve problems. The
>> problems you (Josh) present are real, regardless of Robert's opinion. The
>> thing that is important for everyone to remember is PERCEPTION IS REALITY.
>>
>> If people PERCEIVE there is a problem, THERE IS A PROBLEM.
> Absolutely.  And I am perfectly well aware that we have screwed this
> up from time to time.  But I also know that I have spent a very large
> amount of time over the last few years trying to improve things.  It
> would be nice to know whether that has had any impact.  If it hasn't,
> then half of what I have spent the last two years doing has been a
> waste of time.
I don't think anyone would argue that your efforts have not improved the 
situation. I certainly wouldn't. However, the perception (and reality of 
the problem) definitely still applies. I wouldn't suggest that you stop 
what you are doing but that doesn't mean the problem or variances of the 
problem don't still exist and need to be addressed.

Sincerely,

jD




-- 
Command Prompt, Inc. - http://www.commandprompt.com/
PostgreSQL Support, Training, Professional Services and Developement
Organizers of the PostgreSQL Conference - http://www.postgresqlconference.org/
@cmdpromptinc - @postgresconf - 509-416-6579



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Peter Eisentraut
Date:
On Wed, 2011-04-20 at 16:25 -0400, Tom Lane wrote:
> But ignoring scheduling difficulties, my point here is that
> it seems like the shorter the cycle, the better, for a lot of
> purposes.  Can we do any better than once-a-month, or is that the
> limit given that people need flexible schedules within the fest?

If you want to keep the basic idea of predictable periods of activity
and rest, I think that's as far as you can go.

I'm personally not terribly tied to that; I'm more interested in the
tool support that the CF gives us.  I might also like, for example, just
a permanent patch queue with patches sorted by date.  Multiple
approaches like that could also very well exist in parallel, even within
the existing commitfest application framework.




Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Alvaro Herrera
Date:
Excerpts from Robert Haas's message of mié abr 20 16:22:24 -0300 2011:

> > If people PERCEIVE there is a problem, THERE IS A PROBLEM.
> 
> Absolutely.  And I am perfectly well aware that we have screwed this
> up from time to time.  But I also know that I have spent a very large
> amount of time over the last few years trying to improve things.  It
> would be nice to know whether that has had any impact.  If it hasn't,
> then half of what I have spent the last two years doing has been a
> waste of time.

It may very well be fixed, but if the guys doing the submission (or,
more precisely failing to do it) don't know that things have changed,
they will continue to avoid submitting stuff.

-- 
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Wed, Apr 20, 2011 at 3:42 PM, Josh Berkus <josh@agliodbs.com> wrote:
> On 4/20/11 12:35 PM, Tom Lane wrote:
>> Well, no, that's not the whole story.  To me, what the above idea
>> implies is shifting more of the burden of fixing up patches away from
>> the committer and back to the patch author.  Instead of spending time
>> fixing up not-quite-ready patches myself, I'd be much more ready to
>> tell the patch author "do X, Y, and Z, and come back next month".
>
> Yes, definitely!  For that matter, booting a patch which got no review
> is less of a problem if we're only booting it for 3 weeks.
>
> The whole purpose of the CFs was not to help submitters -- it was to
> help reviewers.   If we just wanted to help submitters, we'd do
> Continuous Integration, and review all the time.  But the reviewers need
> "time off".
>
> I think we should try this for 9.2.  Given the accumulation between then
> and now, I think the first CF should be 2 weeks, and then we can move to
> monthly/weeklong CFs after that.  So it would look like:
>
> CF1: July 16-31
> CF2: August 1-7
> CF3: September 1-7
> CF4: October 1-7
> CF5: November 1-7
> CF6: December 1-7
> CF7: January 3-10
> CF8: February until done

I am concerned that this will get us back into the land of the
interminable last CommitFest.  I believe that one of the reasons why
things didn't go as smoothly before we had the CommitFest was because
patches didn't get dealt with until the end of the cycle.  I think
that if, as proposed, we are faster about pushing patches back on the
submitters when they're not up to snuff, then we will end up having
more stuff bounce along for many CommitFests without actually getting
committed, which will tend to exacerbate the pile-up at the end of the
cycle.  The basic underlying problem here is that there is tremendous
reluctance to boot anything when it means pushing it out to the next
release, and I think that's just terrible project management.  If we
had punted collations and sync rep to 9.2, we would be on beta2 right
now, instead of still trying to get things squared away for beta1.  If
we allow people to submit patches up until supposed feature freeze - 7
days instead of proposed feature freeze - 31 days, that's not going to
help.

Now, maybe if we branched the tree immediately after the last CF of
the release and continued having week-long CFs, we might be able to
make it work.  Then, at least if you didn't get your stuff committed
to the right release, you could still get it committed somewhere.  But
even then I think we'd have this problem of people being unwilling to
give up on jamming stuff into a release, regardless of the scheduling
impact of doing so.  I actually think the problem of getting releases
out on time is a *much* bigger problem for us than how long or short
CommitFests are.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Wed, Apr 20, 2011 at 7:00 PM, Alvaro Herrera
<alvherre@commandprompt.com> wrote:
> Excerpts from Robert Haas's message of mié abr 20 16:22:24 -0300 2011:
>
>> > If people PERCEIVE there is a problem, THERE IS A PROBLEM.
>>
>> Absolutely.  And I am perfectly well aware that we have screwed this
>> up from time to time.  But I also know that I have spent a very large
>> amount of time over the last few years trying to improve things.  It
>> would be nice to know whether that has had any impact.  If it hasn't,
>> then half of what I have spent the last two years doing has been a
>> waste of time.
>
> It may very well be fixed, but if the guys doing the submission (or,
> more precisely failing to do it) don't know that things have changed,
> they will continue to avoid submitting stuff.

Yep.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
tomas@tuxteam.de
Date:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wed, Apr 20, 2011 at 11:39:47AM -0700, Josh Berkus wrote:

[...]

> Review of design concepts and WIP patches has *always* been a problem
> for this project [...]

> We tell people to submit a design concept, but then such submissions are
> often ignored.  When they're not ignored, they often are subject to
> either extreme bikeshedding or a lot of negativity around things the
> author hasn't implemented yet ... even if the author warns that they're
> not implemented.

I'm not a committer. So take this data point for what it's worth. But I
have been following this list for quite a while, and I must say: I (very
respectfully!) disagree. Having  watched mailing lists for other
projects, the quality of the answers one gets here is outstanding. The
tone might be sometimes a bit tight (but never disrespectful or
flaming), but seriously: what do I get off a friendly answer if there is
no content?

The same goes to -GENERAL. I've always got answers to my (sometimes, in
hindsight quite stupid) questions which actually *helped* to solve my
problem.

It's OK to strive to improve the process, but I think you all are quite
good.

[...]

> So in the spirit of NOT reinventing the wheel: ReviewBoard.  Yes,
> really [...]
> [...]  But I think it's time to try something else, maybe
> several other things.

Maybe. But I *do* understand the unwillingness to change that. I've
contributed (tiny) patches to more that one project, and it's
frustrating to fight the bug-tracker-du-jour system. This one won't talk
to me unless my browser talks Javascript. That one... (you get the
idea). I strongly appreciate the free-flowing mailing list style here
(maybe it's just an age problem ;-)

Regards
- -- tomás
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFNr9IhBcgs9XrR2kYRAkw+AJoDFJcnpR06VpGNVAzsbx/eZpQcxACfUv//
vFsZsPiYlM78fxsjCLQvbHw=
=A+7H
-----END PGP SIGNATURE-----


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Peter Eisentraut
Date:
On Wed, 2011-04-20 at 21:09 -0400, Robert Haas wrote:
> But
> even then I think we'd have this problem of people being unwilling to
> give up on jamming stuff into a release, regardless of the scheduling
> impact of doing so.  I actually think the problem of getting releases
> out on time is a *much* bigger problem for us than how long or short
> CommitFests are.

I think to really address that problem, you need to think about shorter
release cycles overall, like every 6 months.  Otherwise, the current 12
to 14 month horizon is just too long psychologically.



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Simon Riggs
Date:
On Wed, Apr 20, 2011 at 8:54 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Peter Eisentraut <peter_e@gmx.net> writes:
>> I would imagine one commit fest per month, but
>> it's only a week long.
>
> BTW, just as a thought experiment: what about a one-day CF once a week?
> "Patch Tuesdays", if you will.  Spend all day reviewing/committing,
> bounce back whatever is not ready, patch authors try again next week.
>
> Really large patches are not going to fit into that paradigm, probably,
> but an awful lot of stuff would --- and it might help encourage more
> incremental development of the big ones, too.

I'm responding to this post with mostly general comments, not directed
specifically at Tom.

Speeding up the process means that people with more time get a bigger
say and people with less time get a smaller input than before. I'm
already concerned that the gap between patch submission and patch
commit is so short it effectively means feedback is impossible.

The more frequently we do integration, the greater proportion of our
time is spent doing that.

My concern is there are a relatively low number of people working on
features that lots of people care about. Senior time should not be
wasted on endless integration.

We should be encouraging people to spend more time on more useful
features, not an endless stream of trivial patches, integration and
release processes. None of our users give a flying, err, squirrel,
about our small patch review process. Especially when its absolutely
brilliant already.

My model of contributing to this project has always been to spend time
with customers, understanding solutions and problems, then bringing
that back to the community. That has brought both the funding to allow
me to contribute and a stream of ideas with a clear focus. I encourage
others to do the same. I don't think we should be working on an
interrupt driven model, we should be planning our contributions and
making sure we make the biggest impact possible with real code, not
just twittering about it constantly. If we spend too much time with
each other we will be exactly like the larger commercial development
groups who never meet users only each other. Even the General list
isn't fully representative of the actual/potential user base.

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


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
"Kevin Grittner"
Date:
Peter Eisentraut <peter_e@gmx.net> wrote:
> you need to think about shorter release cycles overall, like every
> 6 months.
With the current time between feature freeze and release, that
wouldn't leave a lot of time for development.
-Kevin


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Peter Eisentraut
Date:
On Thu, 2011-04-21 at 14:01 +0100, Simon Riggs wrote:
> We should be encouraging people to spend more time on more useful
> features, not an endless stream of trivial patches, integration and
> release processes.

Hence the proposal to cut that time down and make it count better.

Which direction were you thinking?




Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Peter Eisentraut
Date:
On Thu, 2011-04-21 at 08:42 -0500, Kevin Grittner wrote:
> > you need to think about shorter release cycles overall, like every
> > 6 months.
>  
> With the current time between feature freeze and release, that
> wouldn't leave a lot of time for development.

Presumably, one would aim to cut all the other things in half as well.



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Thu, Apr 21, 2011 at 2:43 AM, Peter Eisentraut <peter_e@gmx.net> wrote:
> On Wed, 2011-04-20 at 21:09 -0400, Robert Haas wrote:
>> But
>> even then I think we'd have this problem of people being unwilling to
>> give up on jamming stuff into a release, regardless of the scheduling
>> impact of doing so.  I actually think the problem of getting releases
>> out on time is a *much* bigger problem for us than how long or short
>> CommitFests are.
>
> I think to really address that problem, you need to think about shorter
> release cycles overall, like every 6 months.  Otherwise, the current 12
> to 14 month horizon is just too long psychologically.

I agree.  I am in favor of a shorter release cycle.  But I think that
a shorter release cycle won't work well if there is still four month
long integration period at the end of each series of CommitFests.  The
problem is a bit circular here: because release cycles are long,
people really, really want to slip as much as possible in at the end.
But being under time pressure to get things committed results in a
higher bug count, which means more things that have to be fixed after
feature freeze, which translates into a long release cycle.

I think that it's not too bad if the process of a release getting out
the door results in effectively missing one CommitFest.  For example,
if we imagine one-month CommitFests starting every two months, and we
had a CommitFest starting on January 15th, it wouldn't be too painful
if we skipped a hypothetical March 15th CommitFest to get the release
done, and then started up the process again on May 15th.  However, in
practice, what happens is we miss *two* CommitFests: the expectation
is that the next CommitFest will be on the order of July 15th, which
is just too long.  Similarly, if we did shorter CommitFests and
shorter releases - say, five one-week-a-month CommitFests in July,
August, September, October, and November, I'd want to kick a release
out in December and reopen for development in January, not get stuck
with the same six-month feature freeze we have now, or even a
four-month feature freeze.  But that isn't going to work if people do
the same sort of throwing everything into the kitchen sink at the last
minute that we have been doing for at least the last couple of
releases.

In fact, I don't believe that the current CF cycle really forces a
huge amount of waiting-for-feedback.  It's true that if you submit a
patch at a randomly chosen time, you will have to wait up to two
months for a CommitFest to start, and then you might not get a review
until late in the CommitFest, so it could take you up to three months
to get a review.  In practice, patches are not submitted at random
times - in fact, probably 50% of the patches come in during the last
week before the CF starts, and typically perhaps 50% of the patches
get a review in the first week, and maybe 80% within the first two
weeks.   Some patches also get an initial review between CommitFests,
which further improves the average.  Overall, I bet the average time
between patch submission and first review is <3 weeks.  You can
typically get 2 or 3 followup reviews during the same cycle with only
a few days latency for each.  Even though it would be nice to do
better, for an all-volunteer project, I think it's respectable.   I
can't say the same thing about our process from getting from feature
freeze to release.  It's really long, and it's nearly all fixing bugs
in code that was committed in the last CF, and the last CF produces
exponentially more bugs than the earlier ones, and it's often the case
that people don't fix their own bugs and someone else has to jump in
to pick up the slack.  Meanwhile, the regular flow of reviewing and
committing patches is completely disrupted; and once in a while
someone gets flamed for so much as bringing up a new feature that
they're interested in working on for the next release (which I think
is totally unwarranted; now is the PERFECT time to begin roughing out
plans for 9.2 work... but I digress).

So while I'm mildly interested in the idea of shifting the CF cycle
around to provide more timely review, I can't really get that excited
about it, especially if there's any risk that we are just shifting
more of the work from the CommitFest cycle to the
end-of-release-interminable-integration-period.  However, if there's
some way of avoiding the phenomenon where all hell breaks loose
because people jam four major new features into the tree in as many
weeks, sign me up.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Robert Haas <robertmhaas@gmail.com> writes:
> On Thu, Apr 21, 2011 at 2:43 AM, Peter Eisentraut <peter_e@gmx.net> wrote:
>> I think to really address that problem, you need to think about shorter
>> release cycles overall, like every 6 months. �Otherwise, the current 12
>> to 14 month horizon is just too long psychologically.

> I agree.  I am in favor of a shorter release cycle.

I'm not.  I don't think there is any demand among *users* (as opposed to
developers) for more than one major PG release a year.  It's hard enough
to get people to migrate that often.

Another problem is that if you halve the release interval, you either
double the amount of work spent on maintaining back branches, or halve
the support lifetime of a branch.  Neither of those is attractive.

Now, it certainly would be nice to spend less time in beta mode as
opposed to development, and I think most of the points being made here
are really about how to cut that.  But reducing the release interval is
not going to reduce the total amount of time we spend in beta mode;
in fact I'd expect it to increase.  Halving the amount of development
time per release doesn't mean that you can cut beta time proportionally.
It just takes time to cut a release, and time for testers to try it.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andrew Dunstan
Date:

On 04/21/2011 11:16 AM, Tom Lane wrote:
> Robert Haas<robertmhaas@gmail.com>  writes:
>> On Thu, Apr 21, 2011 at 2:43 AM, Peter Eisentraut<peter_e@gmx.net>  wrote:
>>> I think to really address that problem, you need to think about shorter
>>> release cycles overall, like every 6 months.  Otherwise, the current 12
>>> to 14 month horizon is just too long psychologically.
>> I agree.  I am in favor of a shorter release cycle.
> I'm not.  I don't think there is any demand among *users* (as opposed to
> developers) for more than one major PG release a year.  It's hard enough
> to get people to migrate that often.

I agree.

> Another problem is that if you halve the release interval, you either
> double the amount of work spent on maintaining back branches, or halve
> the support lifetime of a branch.  Neither of those is attractive.

I *really* *really* agree.


cheers

andrew


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
"Ross J. Reedstrom"
Date:
On Thu, Apr 21, 2011 at 11:16:45AM -0400, Tom Lane wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
> > On Thu, Apr 21, 2011 at 2:43 AM, Peter Eisentraut <peter_e@gmx.net> wrote:
> >> I think to really address that problem, you need to think about shorter
> >> release cycles overall, like every 6 months. �Otherwise, the current 12
> >> to 14 month horizon is just too long psychologically.
> 
> > I agree.  I am in favor of a shorter release cycle.
> 
> I'm not.  I don't think there is any demand among *users* (as opposed to
> developers) for more than one major PG release a year.  It's hard enough
> to get people to migrate that often.

In fact, I predict that the observed behavior would be for even more end
users to start skipping releases. Some already do - it's common not to
upgrade unless there's a feature you really need, but for those who do
stay on the 'current' upgrade path, you'll lose some who can't afford to
spend more than one integration-testing round a year.

Ross
-- 
Ross Reedstrom, Ph.D.                                 reedstrm@rice.edu
Systems Engineer & Admin, Research Scientist        phone: 713-348-6166
Connexions                  http://cnx.org            fax: 713-348-3665
Rice University MS-375, Houston, TX 77005
GPG Key fingerprint = F023 82C8 9B0E 2CC6 0D8E  F888 D3AE 810E 88F0 BEDE


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Thu, Apr 21, 2011 at 11:16 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
>> On Thu, Apr 21, 2011 at 2:43 AM, Peter Eisentraut <peter_e@gmx.net> wrote:
>>> I think to really address that problem, you need to think about shorter
>>> release cycles overall, like every 6 months.  Otherwise, the current 12
>>> to 14 month horizon is just too long psychologically.
>
>> I agree.  I am in favor of a shorter release cycle.
>
> I'm not.  I don't think there is any demand among *users* (as opposed to
> developers) for more than one major PG release a year.  It's hard enough
> to get people to migrate that often.

I agree there's probably little user demand, and back-branch
maintenance is an issue, but I think if it removed the temptation to
cram major new features into the tree at the last minute, it might be
worth it.  However, a possibly more likely outcome is that we'd still
have that temptation, just more frequently; and end up with even less
of the year open to new patches than is currently the case.

> Another problem is that if you halve the release interval, you either
> double the amount of work spent on maintaining back branches, or halve
> the support lifetime of a branch.  Neither of those is attractive.
>
> Now, it certainly would be nice to spend less time in beta mode as
> opposed to development, and I think most of the points being made here
> are really about how to cut that.  But reducing the release interval is
> not going to reduce the total amount of time we spend in beta mode;
> in fact I'd expect it to increase.  Halving the amount of development
> time per release doesn't mean that you can cut beta time proportionally.
> It just takes time to cut a release, and time for testers to try it.

I believe that the problem is much more related to the fact that we
commit things at the end of the cycle that aren't really done than it
is to the amount of time beta testers need to try things.  If we were
only waiting on testing, we could branch the tree and call the release
du jour beta for another N months, then release, meanwhile continuing
development.  In fact, you and I and three or four other people have
spent most of our visible PG time over the last 2 months fixing MANY
bugs, mostly in the six or so major features committed between
February 7th and March 6th.  (By way of comparison, notice how few
bugs that have been in the major patches from CF3 - because those
things were actually pretty much working *when they were committed*.)

Now, we're getting to the point where that might actually be a
reasonable way to go.  It wouldn't bother me a bit to branch the tree
just after beta1 and start a new cycle of CommitFests on May 15th, and
we could begin integrating some of the big stuff that didn't make it
into 9.1: key locks, range types, additional sync rep modes, snapshot
cloning, parallel pg_dump, etc.  It would be great to start working on
that stuff while it's still mildly fresh in people's minds, and at the
*beginning* of the release cycle.  We're probably doomed to another
fall release at this point anyway, so it's not clear to me that the
inevitable loss of focus that will ensue is really costing anything.
Had we gotten to beta1 on March 1st, I'd probably be in favor of going
all in to get the release out in June or maybe on July 1, but at this
point that seems unlikely to be realistic.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Thu, Apr 21, 2011 at 11:37 AM, Ross J. Reedstrom <reedstrm@rice.edu> wrote:
> On Thu, Apr 21, 2011 at 11:16:45AM -0400, Tom Lane wrote:
>> Robert Haas <robertmhaas@gmail.com> writes:
>> > On Thu, Apr 21, 2011 at 2:43 AM, Peter Eisentraut <peter_e@gmx.net> wrote:
>> >> I think to really address that problem, you need to think about shorter
>> >> release cycles overall, like every 6 months.  Otherwise, the current 12
>> >> to 14 month horizon is just too long psychologically.
>>
>> > I agree.  I am in favor of a shorter release cycle.
>>
>> I'm not.  I don't think there is any demand among *users* (as opposed to
>> developers) for more than one major PG release a year.  It's hard enough
>> to get people to migrate that often.
>
> In fact, I predict that the observed behavior would be for even more end
> users to start skipping releases. Some already do - it's common not to
> upgrade unless there's a feature you really need, but for those who do
> stay on the 'current' upgrade path, you'll lose some who can't afford to
> spend more than one integration-testing round a year.

Well, that aspect of the problem doesn't bother me, much.  I don't
really care whether people upgrade to each new release the moment it
comes out anyway.  It would require us to keep any
backward-compatibility hacks around for more releases, but we're
pretty good about that anyway.  8.3 broke the world, but the last few
releases have been pretty smooth for most people, I think.

Not to say that there aren't OTHER problems with the idea...

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
[ another thought on this topic ]

Robert Haas <robertmhaas@gmail.com> writes:
> I think that it's not too bad if the process of a release getting out
> the door results in effectively missing one CommitFest. ...
> But that isn't going to work if people do
> the same sort of throwing everything into the kitchen sink at the last
> minute that we have been doing for at least the last couple of
> releases.

> In fact, I don't believe that the current CF cycle really forces a
> huge amount of waiting-for-feedback.  It's true that if you submit a
> patch at a randomly chosen time, you will have to wait up to two
> months for a CommitFest to start, and then you might not get a review
> until late in the CommitFest, so it could take you up to three months
> to get a review.  In practice, patches are not submitted at random
> times - in fact, probably 50% of the patches come in during the last
> week before the CF starts, and typically perhaps 50% of the patches
> get a review in the first week, and maybe 80% within the first two
> weeks.

But aren't those two sides of the same coin, ie, people's natural
tendency to work to a deadline?  If you approve of a lot of patches
showing up just in time for a commitfest, why don't you approve of
big patches showing up just in time for a release?  I mean, I've been
heard to complain about that too, but complaining hasn't changed
anyone's behavior and it's foolish to expect that it will in the
future.  (See insanity, definition of.)

We need to find a way to work with that behavior, not try to change it.
I don't know what exactly.

One idea that comes to mind is to give up on the linear development-mode-
then-beta-mode management model, ie, allow development of release N+1
to start while beta is still going on for release N.  The principal
objection to this in the past has been that the PG development community
is too small to do more than one thing at once, but maybe that's not
true anymore.  The thing I'd be most worried about is how we get enough
energy directed at the release-stabilization part of the work, when for
most developers the new-development part is much more interesting/fun.
But we have that problem in some form already --- it's not clear to me
how much of the community really engages in what happens during beta,
rather than quietly working on stuff for the next release.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andres Freund
Date:
On Thursday, April 21, 2011 05:43:16 PM Robert Haas wrote:
> On Thu, Apr 21, 2011 at 11:37 AM, Ross J. Reedstrom <reedstrm@rice.edu> 
wrote:
> > On Thu, Apr 21, 2011 at 11:16:45AM -0400, Tom Lane wrote:
> >> Robert Haas <robertmhaas@gmail.com> writes:
> >> > I agree.  I am in favor of a shorter release cycle.
> >> I'm not.  I don't think there is any demand among *users* (as opposed to
> >> developers) for more than one major PG release a year.  It's hard enough
> >> to get people to migrate that often.
> > In fact, I predict that the observed behavior would be for even more end
> > users to start skipping releases. Some already do - it's common not to
> > upgrade unless there's a feature you really need, but for those who do
> > stay on the 'current' upgrade path, you'll lose some who can't afford to
> > spend more than one integration-testing round a year.
> Well, that aspect of the problem doesn't bother me, much.  I don't
> really care whether people upgrade to each new release the moment it
> comes out anyway.
> Not to say that there aren't OTHER problems with the idea...
One could argue that its causing bad PR for postgres. I have seen several 
parties planning to migrate away or not migrate to postgres because of 
performance evaluations they made. With 7.4, 8.0 and 8.2. In 2010.

Andres



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Thu, Apr 21, 2011 at 11:46 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> But aren't those two sides of the same coin, ie, people's natural
> tendency to work to a deadline?  If you approve of a lot of patches
> showing up just in time for a commitfest, why don't you approve of
> big patches showing up just in time for a release?  I mean, I've been
> heard to complain about that too, but complaining hasn't changed
> anyone's behavior and it's foolish to expect that it will in the
> future.  (See insanity, definition of.)

Well, I guess I approve of the first behavior because it doesn't feel
like having a red-hot iron spike driven through my foot, and I
disapprove of the second one because it does.  That may not be
entirely consistent taken in the abstract, but it has some solid
practical roots.

> We need to find a way to work with that behavior, not try to change it.
> I don't know what exactly.
>
> One idea that comes to mind is to give up on the linear development-mode-
> then-beta-mode management model, ie, allow development of release N+1
> to start while beta is still going on for release N.  The principal
> objection to this in the past has been that the PG development community
> is too small to do more than one thing at once, but maybe that's not
> true anymore.  The thing I'd be most worried about is how we get enough
> energy directed at the release-stabilization part of the work, when for
> most developers the new-development part is much more interesting/fun.
> But we have that problem in some form already --- it's not clear to me
> how much of the community really engages in what happens during beta,
> rather than quietly working on stuff for the next release.

I totally agree.  In fact, I think that trying to close off that
activity is one of the most self-destructive things we could possibly
do.  It makes missing the release far more painful if you're thinking
about not only a 12-month slip on GA but also a 6-month slip on any
meaningful further review.  Encouraging people to hold off major
proposals for the next release while we are focusing on beta also
tends to slow them down, which then exacerbates the pile-up at the end
of the release cycle.  I would like to blow the doors on that wide
open and encourage people to start submitting design proposals for 9.2
NOW.  NOW, NOW, NOW!  Not in July!  And *really* not next January!
And frankly, the sooner we can realistically start working on
integrating the code that has *already* been written for 9.2, the
better.  The patches are going to land on us at some point, and
dealing with them earlier will allow those people to move on to other
things (which is good), reduce the pile-up at the end of the cycle
(even better), or possibly both.

I'm willing to make a serious commitment to being involved in the
release stabilization work and to give it some degree of priority over
new patches, if that's what it takes to make the process work
smoothly.  We are fundamentally resource-constrained, and no process
is going to change that unless the process change, of itself, causes
more people to contribute more time.  But even if the first CommitFest
involves a slightly higher bounce rate due to lack of
reviewer/committer bandwidth, it's still better than not having one.
There have been maybe half a dozen people who have been principally
responsible for the stabilization that we have done since CF4, and the
community is much larger than that.  Everyone else is either doing
nothing (which is bad), or working without on-list discussion (which
is also bad).  Even for the people who are deeply committed to release
stabilization would probably be happier and more motivated to continue
contributing if they weren't being limited to ONLY that.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Thu, Apr 21, 2011 at 11:48 AM, Andres Freund <andres@anarazel.de> wrote:
> On Thursday, April 21, 2011 05:43:16 PM Robert Haas wrote:
>> On Thu, Apr 21, 2011 at 11:37 AM, Ross J. Reedstrom <reedstrm@rice.edu>
> wrote:
>> > On Thu, Apr 21, 2011 at 11:16:45AM -0400, Tom Lane wrote:
>> >> Robert Haas <robertmhaas@gmail.com> writes:
>> >> > I agree.  I am in favor of a shorter release cycle.
>> >> I'm not.  I don't think there is any demand among *users* (as opposed to
>> >> developers) for more than one major PG release a year.  It's hard enough
>> >> to get people to migrate that often.
>> > In fact, I predict that the observed behavior would be for even more end
>> > users to start skipping releases. Some already do - it's common not to
>> > upgrade unless there's a feature you really need, but for those who do
>> > stay on the 'current' upgrade path, you'll lose some who can't afford to
>> > spend more than one integration-testing round a year.
>> Well, that aspect of the problem doesn't bother me, much.  I don't
>> really care whether people upgrade to each new release the moment it
>> comes out anyway.
>> Not to say that there aren't OTHER problems with the idea...
> One could argue that its causing bad PR for postgres. I have seen several
> parties planning to migrate away or not migrate to postgres because of
> performance evaluations they made. With 7.4, 8.0 and 8.2. In 2010.

That's certainly true.  It's clearly insane to benchmark with anything
other than the latest major release - on any product - if you want to
have any pretense of fairness.  However, for users who have
applications that work and perform acceptably, I don't think it
benefits us to be too aggressive in trying to get them onto a later
major release.  If we wanted to do that, we could maintain
back-branches for two years instead of five, but I don't think that
would be doing anyone any favors.

In fact, I've been wondering if we shouldn't consider extending the
support window for 8.2 past the currently-planned December 2011.
There seem to be quite a lot of people running that release precisely
because the casting changes in 8.3 were so painful, and I think the
incremental effort on our part to extend support for another year
would be reasonably small.  I guess the brunt of the work would
actually fall on the packagers.  It looks like we've done 5 point
releases of 8.2.x in the last year, so presumably if we did decide to
extend the EOL date by a year or so that's about how much incremental
effort would be needed.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Christopher Browne
Date:
On Thu, Apr 21, 2011 at 11:48 AM, Andres Freund <andres@anarazel.de> wrote:
> One could argue that its causing bad PR for postgres. I have seen several
> parties planning to migrate away or not migrate to postgres because of
> performance evaluations they made. With 7.4, 8.0 and 8.2. In 2010.

Well evaluating based on things past that can't be changed in the
absence of time machines doesn't offer us much guidance, as there
isn't anything that can be done in the present to fix such.
-- 
When confronted by a difficult problem, solve it by reducing it to the
question, "How would the Lone Ranger handle this?"


EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Tom Lane
Date:
[ man, this thread has totally outlived its title, could we change that? I'll start with this subtopic ]

Robert Haas <robertmhaas@gmail.com> writes:
> In fact, I've been wondering if we shouldn't consider extending the
> support window for 8.2 past the currently-planned December 2011.
> There seem to be quite a lot of people running that release precisely
> because the casting changes in 8.3 were so painful, and I think the
> incremental effort on our part to extend support for another year
> would be reasonably small.  I guess the brunt of the work would
> actually fall on the packagers.  It looks like we've done 5 point
> releases of 8.2.x in the last year, so presumably if we did decide to
> extend the EOL date by a year or so that's about how much incremental
> effort would be needed.

I agree that the incremental effort would not be so large, but what
makes you think that the situation will change given another year?
My expectation is that'd just mean people will do nothing about
migrating for a year longer.

More generally: it took a lot of argument to establish the current EOL
policy, and bending it the first time anyone feels any actual pain
will pretty much destroy the whole concept.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andres Freund
Date:
On Thursday, April 21, 2011 06:39:44 PM Robert Haas wrote:
> On Thu, Apr 21, 2011 at 11:48 AM, Andres Freund <andres@anarazel.de> wrote:
> > On Thursday, April 21, 2011 05:43:16 PM Robert Haas wrote:
> >> On Thu, Apr 21, 2011 at 11:37 AM, Ross J. Reedstrom <reedstrm@rice.edu>
> > 
> > wrote:
> >> > On Thu, Apr 21, 2011 at 11:16:45AM -0400, Tom Lane wrote:
> >> >> Robert Haas <robertmhaas@gmail.com> writes:
> >> >> > I agree.  I am in favor of a shorter release cycle.
> >> >> 
> >> >> I'm not.  I don't think there is any demand among *users* (as opposed
> >> >> to developers) for more than one major PG release a year.  It's hard
> >> >> enough to get people to migrate that often.
> >> > 
> >> > In fact, I predict that the observed behavior would be for even more
> >> > end users to start skipping releases. Some already do - it's common
> >> > not to upgrade unless there's a feature you really need, but for
> >> > those who do stay on the 'current' upgrade path, you'll lose some who
> >> > can't afford to spend more than one integration-testing round a year.
> >> 
> >> Well, that aspect of the problem doesn't bother me, much.  I don't
> >> really care whether people upgrade to each new release the moment it
> >> comes out anyway.
> >> Not to say that there aren't OTHER problems with the idea...
> > 
> > One could argue that its causing bad PR for postgres. I have seen several
> > parties planning to migrate away or not migrate to postgres because of
> > performance evaluations they made. With 7.4, 8.0 and 8.2. In 2010.
> 
> That's certainly true.  It's clearly insane to benchmark with anything
> other than the latest major release - on any product - if you want to
> have any pretense of fairness.
The usual argument against that is that $version is the only available on 
$platform in version $version...

And I doubt that a higher number of new pg versions will lead to more 
supported releases in distributions...

Andres


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Dave Page
Date:
On Thu, Apr 21, 2011 at 5:59 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> [ man, this thread has totally outlived its title, could we change that?
>  I'll start with this subtopic ]
>
> Robert Haas <robertmhaas@gmail.com> writes:
>> In fact, I've been wondering if we shouldn't consider extending the
>> support window for 8.2 past the currently-planned December 2011.
>> There seem to be quite a lot of people running that release precisely
>> because the casting changes in 8.3 were so painful, and I think the
>> incremental effort on our part to extend support for another year
>> would be reasonably small.  I guess the brunt of the work would
>> actually fall on the packagers.  It looks like we've done 5 point
>> releases of 8.2.x in the last year, so presumably if we did decide to
>> extend the EOL date by a year or so that's about how much incremental
>> effort would be needed.
>
> I agree that the incremental effort would not be so large, but what
> makes you think that the situation will change given another year?
> My expectation is that'd just mean people will do nothing about
> migrating for a year longer.
>
> More generally: it took a lot of argument to establish the current EOL
> policy, and bending it the first time anyone feels any actual pain
> will pretty much destroy the whole concept.

It would also make at least one packager very unhappy as the 8.2
Windows build is by far the hardest and most time consuming to do and
I happen to know he's been counting the days until it goes.

More generally, keeping it for longer means we might end up supporting
6 major releases at once. That may not be so much work on a day to day
basis, but it adds up to a lot at release times, which was one of the
reasons why we agreed on the 5 year support window.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Christopher Browne
Date:
On Thu, Apr 21, 2011 at 11:05 AM, Robert Haas <robertmhaas@gmail.com> wrote:
> I agree.  I am in favor of a shorter release cycle.  But I think that
> a shorter release cycle won't work well if there is still four month
> long integration period at the end of each series of CommitFests.  The
> problem is a bit circular here: because release cycles are long,
> people really, really want to slip as much as possible in at the end.
> But being under time pressure to get things committed results in a
> higher bug count, which means more things that have to be fixed after
> feature freeze, which translates into a long release cycle.

If we somehow were able to come up with a 6 week release cycle, we'd
still have the problem that there are features that take more than 6
weeks to integrate into a release.  (HOT and SyncRep, I'm looking at
you!) Any such larger features would "blow this up," quite forcibly.

I don't think our release cycle is vastly too long; it takes enough
time to plan upgrades for systems that my colleagues at Afilias aren't
keen on using every PG release in production that comes out as it
stands now.

Peter Eisentraut points out that with the way things are, now,  "...
you are left with all of about 20 days per year for discussion,
collaborative planning and coding.  Which is obviously silly, which is
why the process breaks down."

I think the CommitFests have been a *super* tool for addressing such
problems as:
- patches getting lost
- getting review effort put onto the easier patches

But they aren't the only thing we conceptually need to have.  For
tougher features, they're not great.  And they're completely useless
at addressing discussions surrounding things we know we want done, but
don't have a strategy for yet.  Those things aren't "patches", there's
nothing yet to commit.

My sense is that something else is needed as a process to help with
those "nebulous large changes."  I'm not sure quite what it looks
like.  Maybe there's some tooling that would be helpful, but we really
need some experimentation to figure out what the process should look
like.
--
When confronted by a difficult problem, solve it by reducing it to the
question, "How would the Lone Ranger handle this?"


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Josh Berkus
Date:
All,

>>> In fact, I've been wondering if we shouldn't consider extending the
>>> support window for 8.2 past the currently-planned December 2011.
>>> There seem to be quite a lot of people running that release precisely
>>> because the casting changes in 8.3 were so painful, and I think the
>>> incremental effort on our part to extend support for another year
>>> would be reasonably small.  I guess the brunt of the work would
>>> actually fall on the packagers.  It looks like we've done 5 point
>>> releases of 8.2.x in the last year, so presumably if we did decide to
>>> extend the EOL date by a year or so that's about how much incremental
>>> effort would be needed.

Better that someone should just focus on whipping Robert's (or was it
Greg's?) replace-the-missing-casts package into shape as an extension.

I'm sure some kind of corporate sponsorship would be available for this
if someone wanted to work on it.  Enough companies are facing this as
upgrade pain to want to fix it.  If someone wants to work on it, let me
know; I'll start a fundraising campaign.

-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Kenneth Marshall
Date:
On Thu, Apr 21, 2011 at 06:04:09PM +0100, Dave Page wrote:
> On Thu, Apr 21, 2011 at 5:59 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > [ man, this thread has totally outlived its title, could we change that?
> > ?I'll start with this subtopic ]
> >
> > Robert Haas <robertmhaas@gmail.com> writes:
> >> In fact, I've been wondering if we shouldn't consider extending the
> >> support window for 8.2 past the currently-planned December 2011.
> >> There seem to be quite a lot of people running that release precisely
> >> because the casting changes in 8.3 were so painful, and I think the
> >> incremental effort on our part to extend support for another year
> >> would be reasonably small. ?I guess the brunt of the work would
> >> actually fall on the packagers. ?It looks like we've done 5 point
> >> releases of 8.2.x in the last year, so presumably if we did decide to
> >> extend the EOL date by a year or so that's about how much incremental
> >> effort would be needed.
> >
> > I agree that the incremental effort would not be so large, but what
> > makes you think that the situation will change given another year?
> > My expectation is that'd just mean people will do nothing about
> > migrating for a year longer.
> >
> > More generally: it took a lot of argument to establish the current EOL
> > policy, and bending it the first time anyone feels any actual pain
> > will pretty much destroy the whole concept.
> 
> It would also make at least one packager very unhappy as the 8.2
> Windows build is by far the hardest and most time consuming to do and
> I happen to know he's been counting the days until it goes.
> 
> More generally, keeping it for longer means we might end up supporting
> 6 major releases at once. That may not be so much work on a day to day
> basis, but it adds up to a lot at release times, which was one of the
> reasons why we agreed on the 5 year support window.
> 
> -- 
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
> 
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
> 

+1 for cutting the cord on 8.2. People using it still will need
to use the last release available, upgrade, or consult to have
a back-port/build made. 

Regards,
Ken


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Robert Haas
Date:
On Thu, Apr 21, 2011 at 12:59 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> [ man, this thread has totally outlived its title, could we change that?
>  I'll start with this subtopic ]
>
> Robert Haas <robertmhaas@gmail.com> writes:
>> In fact, I've been wondering if we shouldn't consider extending the
>> support window for 8.2 past the currently-planned December 2011.
>> There seem to be quite a lot of people running that release precisely
>> because the casting changes in 8.3 were so painful, and I think the
>> incremental effort on our part to extend support for another year
>> would be reasonably small.  I guess the brunt of the work would
>> actually fall on the packagers.  It looks like we've done 5 point
>> releases of 8.2.x in the last year, so presumably if we did decide to
>> extend the EOL date by a year or so that's about how much incremental
>> effort would be needed.
>
> I agree that the incremental effort would not be so large, but what
> makes you think that the situation will change given another year?
> My expectation is that'd just mean people will do nothing about
> migrating for a year longer.
>
> More generally: it took a lot of argument to establish the current EOL
> policy, and bending it the first time anyone feels any actual pain
> will pretty much destroy the whole concept.

I don't think that's quite a fair description of the proposal.  I
don't think that having a general policy about EOL should preclude us
from making exceptions when there is some particularly compelling
reason to do so, and "it's particularly difficult to upgrade to
release X+1" seems to me to be something that might merit a bit of
consideration in that area.  It is hard to imagine that 8.3, 8.4, 9.0,
or 9.1 could justify special treatment on similar grounds, nor did
7.4, 8.0, or 8.1, which we recently retired under this policy.

However, I can see that I'm way, way in the minority on this one, so
never mind!  It was just a thought...

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Tom Lane
Date:
Dave Page <dpage@pgadmin.org> writes:
> On Thu, Apr 21, 2011 at 5:59 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> I agree that the incremental effort would not be so large, but what
>> makes you think that the situation will change given another year?

> It would also make at least one packager very unhappy as the 8.2
> Windows build is by far the hardest and most time consuming to do and
> I happen to know he's been counting the days until it goes.

Well, if we did extend support for 8.2, we could specifically exclude
Windows.  But I'm still unclear on what would really be accomplished
by extending support for it.  Sooner or later we have to get people
to migrate up from it, and I see no reason to think that supporting
it for just a year more will change anything.
        regards, tom lane


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Tom Lane
Date:
Josh Berkus <josh@agliodbs.com> writes:
> Better that someone should just focus on whipping Robert's (or was it
> Greg's?) replace-the-missing-casts package into shape as an extension.

I think Peter originated that, actually.  My recollection is that there
didn't seem to be any way to extend it to a complete solution, and
besides which it's really a crutch to avoid fixing bugs in your
application.  Still, if someone does want to expend more work on it
I wouldn't object.
        regards, tom lane


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Jaime Casanova
Date:
On Thu, Apr 21, 2011 at 12:35 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> But I'm still unclear on what would really be accomplished
> by extending support for it.  Sooner or later we have to get people
> to migrate up from it, and I see no reason to think that supporting
> it for just a year more will change anything.
>

And people is more likely to migrate if they see some kind of hard
line, specially when migrate means a lot of work.
Actually, someone i know is targeting to migrate before the EOL, just
because the EOL exists.

--
Jaime Casanova         www.2ndQuadrant.com
Professional PostgreSQL: Soporte y capacitación de PostgreSQL


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Greg Stark
Date:
On Thu, Apr 21, 2011 at 6:18 PM, Robert Haas <robertmhaas@gmail.com> wrote:
> However, I can see that I'm way, way in the minority on this one, so
> never mind!  It was just a thought...
>

Fwiw I would have agreed with you on the basic question. Just because
we've said that users can count on N years of support doesn't mean
there's anything binding us to *not* support things for N+x years. The
argument that we should cut refuse to back-patch security fixes and
bug fixes that we could handle without much effort to versions that
users are using just because we think we know better than them and
know they should upgrade is a bad path imho.

However your theory was all predicated on the idea that supporting 8.2
was not much incremental effort and Dave said that's not true so this
is all moot. Doing it Windows-excluded seems not worth the effort ---
unless... what version of Postgres was shipped in the last supported
releases of major distributions? I think it was 8.1 in Ubuntu Hardy
and 8.4 in Ubuntu Lucid so that's irrelevant. What about Redhat and
Debian?

--
greg


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Tom Lane
Date:
Greg Stark <gsstark@mit.edu> writes:
> Fwiw I would have agreed with you on the basic question. Just because
> we've said that users can count on N years of support doesn't mean
> there's anything binding us to *not* support things for N+x years.

Certainly.  The question is what's the point --- and perhaps even more
to the point, if we extend 8.2 support, when are we going to stop
extending it?

> However your theory was all predicated on the idea that supporting 8.2
> was not much incremental effort and Dave said that's not true so this
> is all moot. Doing it Windows-excluded seems not worth the effort ---
> unless... what version of Postgres was shipped in the last supported
> releases of major distributions? I think it was 8.1 in Ubuntu Hardy
> and 8.4 in Ubuntu Lucid so that's irrelevant. What about Redhat and
> Debian?

So far as Red Hat is concerned, neither 8.2 nor 8.3 are of any interest
whatsoever.  I'm still on the hook for 7.4 and 8.1 to some extent, but
only very severe security issues are likely to be considered for those.
        regards, tom lane


Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Andrew Dunstan
Date:

On 04/21/2011 05:17 PM, Greg Stark wrote:
> what version of Postgres was shipped in the last supported
> releases of major distributions? I think it was 8.1 in Ubuntu Hardy
> and 8.4 in Ubuntu Lucid so that's irrelevant. What about Redhat and
> Debian?


IIRC RedHat has a ten year EOL policy, so what they have shipped in old 
releases should not really bind us. In any case, our EOL policy only 
affects what formal releases we make. We can commit fixes to branches 
past their EOL date, and IIRC Tom did this not long ago.

cheers

andrew



Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Peter Eisentraut
Date:
On tor, 2011-04-21 at 13:39 -0400, Tom Lane wrote:
> Josh Berkus <josh@agliodbs.com> writes:
> > Better that someone should just focus on whipping Robert's (or was it
> > Greg's?) replace-the-missing-casts package into shape as an extension.
> 
> I think Peter originated that, actually.  My recollection is that there
> didn't seem to be any way to extend it to a complete solution, and
> besides which it's really a crutch to avoid fixing bugs in your
> application.  Still, if someone does want to expend more work on it
> I wouldn't object.

http://petereisentraut.blogspot.com/2008/03/readding-implicit-casts-in-postgresql.html

There are some problems if you just add *all* the casts back without
thinking.  In particular, the || appears to be causing problems.

But other than those few specific cases, that tool kit fixes all known
problems.  So anyone who is willing to spend more than zero minutes on
planning and executing a major version upgrade shouldn't really have any
problems with this aspect.




Re: EOL for 8.2 (was Re: Formatting Curmudgeons WAS: MMAP Buffers)

From
Peter Eisentraut
Date:
On tor, 2011-04-21 at 22:17 +0100, Greg Stark wrote:
> However your theory was all predicated on the idea that supporting 8.2
> was not much incremental effort and Dave said that's not true so this
> is all moot. Doing it Windows-excluded seems not worth the effort ---
> unless... what version of Postgres was shipped in the last supported
> releases of major distributions? I think it was 8.1 in Ubuntu Hardy
> and 8.4 in Ubuntu Lucid so that's irrelevant. What about Redhat and
> Debian?

Debian: 8.3 in oldstable (<1 year left), 8.4 in stable, probably 9.1 in
next



Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Greg Smith
Date:
On 04/21/2011 12:39 PM, Robert Haas wrote:
> In fact, I've been wondering if we shouldn't consider extending the
> support window for 8.2 past the currently-planned December 2011.
> There seem to be quite a lot of people running that release precisely
> because the casting changes in 8.3 were so painful, and I think the
> incremental effort on our part to extend support for another year
> would be reasonably small.

The pending EOL for 8.2 is the only thing that keeps me sane when 
speaking with people who refuse to upgrade, yet complain that their 8.2 
install is slow.  This last month, that seems to be more than usual "why 
does autovacuum suck so much?" complaints that would all go away with an 
8.3 upgrade.  Extending the EOL is not doing any of these users a 
favor.  Every day that goes by when someone is on a version of 
PostgreSQL that won't ever allow in-place upgrade is just making worse 
the eventual dump and reload they face worse.  The time spent porting to 
8.3 is a one-time thing; the suffering you get trying to have a 2011 
sized database on 2006's 8.2 just keeps adding up the longer you 
postpone it.

-- 
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD
PostgreSQL Training, Services, and 24x7 Support  www.2ndQuadrant.us




Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Greg Smith
Date:
On the big picture of scheduling issues, I have never seen a major piece 
of software ship every 6 months without being incredibly buggy.  I'd 
lose serious faith in this project if that happens here.  Since I've 
never seen a major operating system ship usefully more than about once 
every two years, so I'm not sure who it would be serving anyway.  Even 
if this project pulled it off, those who would see the benefit because 
they're using things like the upgrade-happy Ubuntu/Fedora/Gentoo 
treadmills are clearly not optimizing for the sort of things database 
users care about anyway.  So whacking around the low-level schedule to 
aim at that goal boggles my mind.

As for trying to improve things within the existing yearly cycle, there 
are several types of patch to be concerned about here.  And the useful 
interval to respond isn't the same for all of them.

This discussion started with "newbie patch".  I'd like to see these get 
a review sufficient to say "you're not following the good practices 
outlined by our code guidelines and we can't do anything with this" 
quickly, with a hand-off to resources to help them with that.  Everyone 
reading this list surely knows where that documentation is at now after 
all this publicity.  You might schedule a weekly "answer the newbies" 
scan usefully to help with this.  But the project doesn't get much out 
of that besides being more friendly and encouraging, to help in the 
growing the community long-term.  In the short term, adding more process 
here just to help these submitters will, pragmatically, mainly get in 
the way of working on more finished patches.

Second is "WIP", where the author knows there are issues but is looking 
for feedback.  In the cases where these are interesting to people, these 
sometimes get immediate feedback too.  The ones that don't are because 
a) it's hard to review, or b) no one else is interested enough to poke 
at it immediately.  That means a reviewer will likely need to be either 
assigned or found & motivated to look for it.  And that's painful enough 
that you don't want to do it regularly.  The overhead of herding patch 
reviewers is seriously underestimated by some of the ideas throw around 
here for reducing the intervals of this process.  It's only reasonable 
to do in bulk, where you can at least yelp on-list to try and get 
volunteers usefully.

[There were complaints upthread about things like how Aster's patch 
submissions were treated.  Those were WIP patches that half implemented 
some useful ideas.  But they were presented as completed features, and 
they seemed to expect the community would pick those up and commit in 
that not quite right state without extended additional work on their 
side.  Not doing that sort of thing is part of the reason the PostgreSQL 
code isn't filled with nothing but the fastest hack to get any given job 
done.  Anyone who thinks I'm misrepresenting that view of history should 
revisit the lengthy feedback provided to them at 
https://commitfest.postgresql.org/action/patch_view?id=173 and 
https://commitfest.postgresql.org/action/patch_view?id=205 -- it 
actually goes back even further than that because the first versions of 
these patches were even less suitable for commit.]

Next up is "solid patch that needs technical review".  This is mainly 
different from the WIP case in that it's unlikely any quick feedback 
will help the submitter.  So here it's back to needing to find a 
reviewer again.

Finally, "big feature patch", likely taking multiple CFs to process.  
It's barely possible to get useful feedback on these every two months.   
I don't see how dropping the interval is going to solve any of the 
problems around these.  Two to three people all need to get aligned for 
progress on these to happen:  the author, a reviewer, and a committer, 
with the commiter sometimes also doing the initial review.  Good luck 
making that happen more often than it already does.

I think that anyone who suggests shortening the cycles here, or making 
the CommitFests more frequent, should volunteer to run one.  That will 
beat the idea right out of you.  Work on the problem of how to 
motivate/create more patch reviewers instead; that's where the actual 
bottleneck in this process is at.  Part of the problem with how newbies 
are handled is that they jump right to writing patches, because that's 
cooler to do, rather than starting with doing review.  That's 
counterproductive--best way to learn how to write a good patch is to 
consider the difficulty someone else faces reading one--but you can't 
tell that to some people usefully.

That goes double for some of the people complaining in this thread about 
dissatisfaction with the current process.  If you're not helping review 
patches already, you're not participating in the thing that needs the 
most help.  This is not a problem you make better with fuzzy management 
directives to be nicer to people.  There are real software engineering 
issues about how to ensure good code quality at its core.

-- 
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD
PostgreSQL Training, Services, and 24x7 Support  www.2ndQuadrant.us




Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Bruce Momjian
Date:
Tom Lane wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
> > ... Maybe someone out there is under the impression
> > that I get high off of rejecting patches; but the statistics you cite
> > from the CF app don't exactly support the contention that I'm going
> > around looking for reasons to reject things, or if I am, I'm doing a
> > pretty terrible job finding them.
> 
> Hm ... there are people out there who think *I* get high off rejecting
> patches.  I have a t-shirt to prove it.  But I seem to be pretty
> ineffective at it too, judging from these numbers.

Late reply, but almost all the things Tom rejects I would have rejected
too.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Mon, May 9, 2011 at 10:58 AM, Bruce Momjian <bruce@momjian.us> wrote:
> Tom Lane wrote:
>> Robert Haas <robertmhaas@gmail.com> writes:
>> > ... Maybe someone out there is under the impression
>> > that I get high off of rejecting patches; but the statistics you cite
>> > from the CF app don't exactly support the contention that I'm going
>> > around looking for reasons to reject things, or if I am, I'm doing a
>> > pretty terrible job finding them.
>>
>> Hm ... there are people out there who think *I* get high off rejecting
>> patches.  I have a t-shirt to prove it.  But I seem to be pretty
>> ineffective at it too, judging from these numbers.
>
> Late reply, but almost all the things Tom rejects I would have rejected
> too.

Well, I think I've been guilty more than once of leaning on Tom to try
to get him to accept patches that he might've been inclined to reject.I think that my standards for code quality are
similarto Tom's 
(though sometimes I let through things he would have caught, woops)
but I think I am more inclined to commit feature changes that he might
not find entirely worthwhile.  Like Tom, I'm reasonably wary of random
knickknacks that are extremely special-purpose or will slow down
common cases, but on the average I think I'm slightly more
new-feature-positive than he is.  Not without some exceptions, of
course.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Bruce Momjian
Date:
Greg Smith wrote:
> On 04/21/2011 12:39 PM, Robert Haas wrote:
> > In fact, I've been wondering if we shouldn't consider extending the
> > support window for 8.2 past the currently-planned December 2011.
> > There seem to be quite a lot of people running that release precisely
> > because the casting changes in 8.3 were so painful, and I think the
> > incremental effort on our part to extend support for another year
> > would be reasonably small.
> 
> The pending EOL for 8.2 is the only thing that keeps me sane when 
> speaking with people who refuse to upgrade, yet complain that their 8.2 
> install is slow.  This last month, that seems to be more than usual "why 
> does autovacuum suck so much?" complaints that would all go away with an 
> 8.3 upgrade.  Extending the EOL is not doing any of these users a 
> favor.  Every day that goes by when someone is on a version of 
> PostgreSQL that won't ever allow in-place upgrade is just making worse 
> the eventual dump and reload they face worse.  The time spent porting to 
> 8.3 is a one-time thing; the suffering you get trying to have a 2011 
> sized database on 2006's 8.2 just keeps adding up the longer you 
> postpone it.

Interesting.  You could argue that once 8.3 is our earliest supported
release that we could even shrink the support window because the
argument "I can't dump/reload my data" would be gone.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Greg Smith
Date:
On 05/09/2011 12:06 PM, Andrew Dunstan wrote:
> The fact that we can do in place upgrades of the data only addresses 
> one pain point in upgrading. Large legacy apps require large retesting 
> efforts when upgrading, often followed by lots more work renovating 
> the code for backwards incompatibilities. This can be a huge cost for 
> what the suits see as little apparent gain, and making them do it more 
> frequently in order to stay current will not win us any friends.

I just had a "why a new install on 8.3?" conversation today, and it was 
all about the application developer not wanting to do QA all over again 
for a later release.

Right now, one of the major drivers for "why upgrade?" has been the 
performance improvements in 8.3, relative to any older version.  The 
main things pushing happy 8.3 sites to 8.4 or 9.0 that I see are either 
VACUUM issues (improved with partial vacuum in 8.4) or wanting real-time 
replication (9.0).  I predict many sites that don't want either are 
likely to sit on 8.3 for a really long time.  The community won't be 
able to offer a compelling reason why smaller sites in particular should 
go through the QA an upgrade requires.  The fact that the app QA time is 
now the main driver--not the dump and reload time--is good, because it 
makes it does make it easier for the people with the biggest data sets 
to move.  They're the ones that need the newer versions the most anyway, 
and in that regard having in-place upgrade start showing up as of 8.3 
was really just in time.

I think 8.3 is going to be one of those releases like 7.4, where people 
just keep running it forever.  At least shortening the upgrade path has 
made that concern a little bit better.

-- 
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD
PostgreSQL Training, Services, and 24x7 Support  www.2ndQuadrant.us




Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Robert Haas
Date:
On Mon, May 9, 2011 at 11:25 AM, Bruce Momjian <bruce@momjian.us> wrote:
> Greg Smith wrote:
>> On 04/21/2011 12:39 PM, Robert Haas wrote:
>> > In fact, I've been wondering if we shouldn't consider extending the
>> > support window for 8.2 past the currently-planned December 2011.
>> > There seem to be quite a lot of people running that release precisely
>> > because the casting changes in 8.3 were so painful, and I think the
>> > incremental effort on our part to extend support for another year
>> > would be reasonably small.
>>
>> The pending EOL for 8.2 is the only thing that keeps me sane when
>> speaking with people who refuse to upgrade, yet complain that their 8.2
>> install is slow.  This last month, that seems to be more than usual "why
>> does autovacuum suck so much?" complaints that would all go away with an
>> 8.3 upgrade.  Extending the EOL is not doing any of these users a
>> favor.  Every day that goes by when someone is on a version of
>> PostgreSQL that won't ever allow in-place upgrade is just making worse
>> the eventual dump and reload they face worse.  The time spent porting to
>> 8.3 is a one-time thing; the suffering you get trying to have a 2011
>> sized database on 2006's 8.2 just keeps adding up the longer you
>> postpone it.
>
> Interesting.  You could argue that once 8.3 is our earliest supported
> release that we could even shrink the support window because the
> argument "I can't dump/reload my data" would be gone.

Personally, I think the support window is on the borderline of being
too short already.  There are several Linux distributions out there
that offer 5-year support for certain releases.  Even assuming they
incorporate the latest version of PostgreSQL at the time they wrap the
final release, it'll already be some months since we released that
version, and that means we'll stop supporting that version of
PostgreSQL before they stop supporting that release.  I regularly have
systems that run for 3 or 4 years without needing to be reinstalled,
and they're not necessarily running the bleeding-edge version of
PostgreSQL when first installed.  So they, too, are on the trailing
edge of our support.  As much as I believe that 9.0 (and, now, 9.1)
are the future and people should move to them, we can't enforce that.
EOL doesn't necessarily drive people to move.  If they're just running
"yum update" they're going to get 8.whatever.latest, and that's out of
support and missing relevant bug fixes, then it is.  I haven't run
into much 8.1 recently, but it seems there is still a decent chunk of
8.2 out there.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Bruce Momjian
Date:
Robert Haas wrote:
> > Interesting. ?You could argue that once 8.3 is our earliest supported
> > release that we could even shrink the support window because the
> > argument "I can't dump/reload my data" would be gone.
> 
> Personally, I think the support window is on the borderline of being
> too short already.  There are several Linux distributions out there
> that offer 5-year support for certain releases.  Even assuming they
> incorporate the latest version of PostgreSQL at the time they wrap the
> final release, it'll already be some months since we released that
> version, and that means we'll stop supporting that version of
> PostgreSQL before they stop supporting that release.  I regularly have
> systems that run for 3 or 4 years without needing to be reinstalled,
> and they're not necessarily running the bleeding-edge version of
> PostgreSQL when first installed.  So they, too, are on the trailing
> edge of our support.  As much as I believe that 9.0 (and, now, 9.1)
> are the future and people should move to them, we can't enforce that.
> EOL doesn't necessarily drive people to move.  If they're just running
> "yum update" they're going to get 8.whatever.latest, and that's out of
> support and missing relevant bug fixes, then it is.  I haven't run
> into much 8.1 recently, but it seems there is still a decent chunk of
> 8.2 out there.

I agree we don't want to shorten the window --- I was just pointing out
that we have more upgrade options than in the past.  One big push for
shortening was the Win32 issues on 8.0 and perhaps 8.1 that were
unfixable, which helped push retiring, at least on that platforms, and
once you retire on one platform, there is momentum to retire all
platforms for that release.

With Win32 stable on 8.2, we could say we don't need to shorten the
window as much, but pg_upgrade would allow us to keep it the same as now
because upgrades are potentially easier.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Bruce Momjian
Date:
Greg Smith wrote:
> [There were complaints upthread about things like how Aster's patch 
> submissions were treated.  Those were WIP patches that half implemented 
> some useful ideas.  But they were presented as completed features, and 
> they seemed to expect the community would pick those up and commit in 
> that not quite right state without extended additional work on their 
> side.  Not doing that sort of thing is part of the reason the PostgreSQL 
> code isn't filled with nothing but the fastest hack to get any given job 
> done.  Anyone who thinks I'm misrepresenting that view of history should 
> revisit the lengthy feedback provided to them at 
> https://commitfest.postgresql.org/action/patch_view?id=173 and 
> https://commitfest.postgresql.org/action/patch_view?id=205 -- it 
> actually goes back even further than that because the first versions of 
> these patches were even less suitable for commit.]

[ Again, sorry for my late reply.]

Greg hits a big item above --- it takes 3-4x more work to get a patch to
merge cleanly into our code ("look like it was always there") than to
write the initial patch.  If the author isn't willing to do that 3-4x
work, it is not something the community is going to do on a regular
basis, so it is not surprising the patches are dropped.  This is very
often true of academicly-developed patches too.  (I know I rewrite my
patches 4-5 times, and some feel even that is not enough interations for
me.  ;-) )

> That goes double for some of the people complaining in this thread about 
> dissatisfaction with the current process.  If you're not helping review 
> patches already, you're not participating in the thing that needs the 
> most help.  This is not a problem you make better with fuzzy management 
> directives to be nicer to people.  There are real software engineering 
> issues about how to ensure good code quality at its core.

I agree on this one too.  It is good for people outside the patch review
group to make suggestions (external review is good), but when those
external people can't give clear examples of problems, it is impossible
for the patch review group to react or improve, and the complaints do
more harm than good.  The complaints did spark discussion to reevaluate
our development process, so something good did come out of it.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Tom Lane
Date:
Robert Haas <robertmhaas@gmail.com> writes:
> On Mon, May 9, 2011 at 11:25 AM, Bruce Momjian <bruce@momjian.us> wrote:
>> Interesting. �You could argue that once 8.3 is our earliest supported
>> release that we could even shrink the support window because the
>> argument "I can't dump/reload my data" would be gone.

> Personally, I think the support window is on the borderline of being
> too short already.  There are several Linux distributions out there
> that offer 5-year support for certain releases.

Keep in mind that at least some contributors are paid to do exactly that
long-term support (and if you've not heard, Red Hat is up to seven years
support on RHEL ...).  So the work is going to get done, and if it
doesn't get committed to the community SCM, I'm not sure that really
helps anybody.

Although whether we do formal releases is a different question.  Maybe
it would be sensible to continue patching an old branch but not bother
wrapping up release tarballs?  But the incremental work to do one more
set of release notes and one more tarball build is not that large.
        regards, tom lane


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Bruce Momjian
Date:
Tom Lane wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
> > On Mon, May 9, 2011 at 11:25 AM, Bruce Momjian <bruce@momjian.us> wrote:
> >> Interesting. �You could argue that once 8.3 is our earliest supported
> >> release that we could even shrink the support window because the
> >> argument "I can't dump/reload my data" would be gone.
> 
> > Personally, I think the support window is on the borderline of being
> > too short already.  There are several Linux distributions out there
> > that offer 5-year support for certain releases.
> 
> Keep in mind that at least some contributors are paid to do exactly that
> long-term support (and if you've not heard, Red Hat is up to seven years
> support on RHEL ...).  So the work is going to get done, and if it
> doesn't get committed to the community SCM, I'm not sure that really
> helps anybody.
> 
> Although whether we do formal releases is a different question.  Maybe
> it would be sensible to continue patching an old branch but not bother
> wrapping up release tarballs?  But the incremental work to do one more
> set of release notes and one more tarball build is not that large.

I think the big reason we trimmed the support window was to push people
off of old releases, not to lighten our workload.  Until we stated that
a release was not supported, we didn't give administrators ammunition to
force upgrades within their organizations.

Yeah, that is a lousy reason, but it was the stated case when we shrunk
to five years.  You can argue that our more recent releases are not as
"stop using them" bad as previous ones, but Greg Smith's statement about
autovacuum badness reinforces that goal.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


Re: Formatting Curmudgeons WAS: MMAP Buffers

From
Andrew Dunstan
Date:

On 05/09/2011 11:43 AM, Robert Haas wrote:
>> Interesting.  You could argue that once 8.3 is our earliest supported
>> release that we could even shrink the support window because the
>> argument "I can't dump/reload my data" would be gone.
> Personally, I think the support window is on the borderline of being
> too short already.  There are several Linux distributions out there
> that offer 5-year support for certain releases.

Some (RH?) offer significantly longer periods.

I agree that we should not reduce the support window. The fact that we 
can do in place upgrades of the data only addresses one pain point in 
upgrading. Large legacy apps require large retesting efforts when 
upgrading, often followed by lots more work renovating the code for 
backwards incompatibilities. This can be a huge cost for what the suits 
see as little apparent gain, and making them do it more frequently in 
order to stay current will not win us any friends. I often want to wait 
a while after a release for certain customers, while it beds down, and 
to get them to start moving towards upgrading well before it's the last 
minute. That makes an effective life of four years or less per release 
as things are now. That's plenty short enough.

cheers

andrew