From: | Mark Kirkwood <mark(dot)kirkwood(at)catalyst(dot)net(dot)nz> |
---|---|
To: | Craig Ringer <craig(at)2ndquadrant(dot)com>, Josh Berkus <josh(at)agliodbs(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Joe Conway <mail(at)joeconway(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgreSQL(dot)org> |
Subject: | Re: Commitfest problems |
Date: | 2014-12-14 06:12:36 |
Message-ID: | 548D2A54.3050101@catalyst.net.nz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 13/12/14 22:37, Craig Ringer wrote:
> On 12/12/2014 06:02 AM, Josh Berkus wrote:
>>
>> Speaking as the originator of commitfests, they were *always* intended
>> to be a temporary measure, a step on the way to something else like
>> continuous integration.
>
> I'd really like to see the project revisit some of the underlying
> assumptions that're being made in this discussion:
>
> - Patches must be email attachments to a mailing list
>
> - Changes must be committed by applying a diff
>
> ... and take a look at some of the options a git-based workflow might
> offer, especially in combination with some of the tools out there that
> help track working branches, run CI, etc.
>
> Having grown used to push/pull workflows with CI integration I find the
> PostgreSQL patch workflow very frustrating, especially for larger
> patches. It's particularly annoying to see a patch series squashed into
> a monster patch whenever it changes hands or gets rebased, because it's
> being handed around as a great honking diff not a git working branch.
>
> Is it time to stop using git like CVS?
>
> (/hides)
>
Having dealt with other projects that use a more git centric + CI
approach, I can say that trying to do reviews can be just frustrating in
that case too:
- quirky and annoying web interfaces
- changesets "expiring" in the middle of you reviewing them
- pulls and rebases making actually making it harder to see what was
changed in new changeset versions
I think the basic issue is that reviewing is hard, and while our
system-ismation of the workflow is really primitive, and could be much
better (that seems to be being worked on), the *tool* is not really the
problem.
regards
Mark
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2014-12-14 06:24:13 | Re: pg_basebackup vs. Windows and tablespaces |
Previous Message | Michael Paquier | 2014-12-14 05:28:34 | Re: [REVIEW] Re: Compression of full-page-writes |