From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | Peter Geoghegan <pg(at)bowt(dot)ie> |
Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Pg Committers <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: Tips on committing |
Date: | 2018-06-29 12:23:13 |
Message-ID: | 20180629122313.GC8939@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Thu, Jun 28, 2018 at 10:52:42AM -0700, Peter Geoghegan wrote:
> On Thu, Jun 28, 2018 at 9:52 AM, Alvaro Herrera
> I'll try to do that, but I'd still recommend personalizing it. A lot
> of the stuff in there is specific to my own workflow and tool
> preferences, and my own personal working style. I don't really use a
> template in the same way Bruce does, for example, because most of that
> stuff is taken care of by my text editor having a "gitcommit" syntax.
>
> I find it useful to have a routine checklist for things like
> committing, because it frees up a little space in my head for other
> things. I do the same thing when preparing for a trip.
Yes, that's a good idea. In order to run the tests, I have a dedicated
alias:
alias pgcheck='cd $HOME/postgres && make check-world -j 4 PROVE_FLAGS='\''-j 4'\''
This does not work with 9.5 and older versions though as parallel build
is broken in those cases, and also checking that parallel build is not
broken is a good routine to have in my opinion.
I would also recommend that people use PG_TEST_EXTRA='ssl ldap kerberos'
in their environment so as no tests are skipped. We have seen the SSL
test suite broken silently a couple of times since its introduction...
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Tomas Vondra | 2018-06-29 12:39:34 | Re: Tips on committing |
Previous Message | Michael Paquier | 2018-06-29 12:19:26 | Re: Tips on committing |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2018-06-29 12:30:13 | Re: Remove mention in docs that foreign keys on partitioned tables are not supported |
Previous Message | Alexander Kuzmenkov | 2018-06-29 12:22:04 | Re: Generating partitioning tuple conversion maps faster |