From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Andres Freund <andres(at)anarazel(dot)de>, Magnus Hagander <magnus(at)hagander(dot)net>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, pgsql-committers <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Update copyright for 2017 |
Date: | 2017-01-06 16:14:57 |
Message-ID: | 20170106161457.GA22408@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Wed, Jan 4, 2017 at 10:05:10AM -0500, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
> > On 2017-01-03 13:02:28 -0500, Bruce Momjian wrote:
> >> Yeah, I was doing parallel pulls of different branches in git via shell
> >> script, and it seems the size of this commit showed me that doesn't
> >> work. Sorry.
>
> > Shouldn't you check the results of something like this before pushing?
> > Sorry for piling on, but that seems like a quite critical step.
>
> Actually, my takeaway from this was "don't ever use git reset on the repo".
> "git revert" would have been much safer. Yeah, it would have meant that
> git blame on the 9.2 branch would have some useless noise, but how much
> does anyone still care about that?
FYI, my takeaway is that you can't run parallel 'git pull' operations on
different branches checked out of the same tree, at least as setup from
instructions on our wiki. Such parallel operations generate messages
indicating someone else is working on the tree, but I now realize those
are _warnings_ and this behavior should be avoided. Normally it works
fine, but for a big commit, it can move a changeset from one branch to
another, with disastrous consequences.
Yes, I should have checked more, but I often do a pull before a push as
another check, so that can mess it up too. Basically, if your tools are
being used improperly, checking can make things worse too.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-01-06 19:13:09 | pgsql: Invalidate cached plans on FDW option changes. |
Previous Message | Robert Haas | 2017-01-06 14:38:51 | pgsql: Repair commit b81b5a96f424531b97cdd1dba97d9d1b9c9d372e. |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2017-01-06 16:16:26 | Re: Performance degradation in Bitmapscan (commit 75ae538bc3168bf44475240d4e0487ee2f3bb376) |
Previous Message | Andres Freund | 2017-01-06 16:06:17 | Re: Performance degradation in Bitmapscan (commit 75ae538bc3168bf44475240d4e0487ee2f3bb376) |