Mailing-List: contact dev-help@ignite.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.incubator.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CAGcMBHg7hsn8qQWhRsBSpzG8=USLVP4TbD_usLEQwYwnp7iWNA@mail.gmail.com>
References: 
 <CAGcMBHh3PSvt9HXGG8kb3kU07zvnqOKaS97aVytZ2X6pyG+62w@mail.gmail.com>
 <20150706190038.GH15767@tpx>
 <CABDss3hXbO8uU6NR7r5pmZhJLPgEXCgpWHsQGGkOMLWbOqMbzw@mail.gmail.com>
 <20150709200028.GM28615@boudnik.org> <20150713184505.GX28615@boudnik.org>
 <CABDss3i_ayhqX2g5mwjZt48BAgzQGdZ+X2uqrhywT=19CFnTGA@mail.gmail.com>
 <CA+0=VoVkd6ivDPa8rNc8m-kXtcLZM8=6tbdHb3cnWU7q-rjv5Q@mail.gmail.com>
 <CABDss3iwLd1Vn1Nr4ZFGTybjMFLT=FDqvrJE3pFKeV_S=M6kEw@mail.gmail.com>
 <CA+0=VoUDc1uDOnz9hDa176wTLHmQ0rT_Qihmh3L9JuW2nnc0PQ@mail.gmail.com>
 <20150727210102.GV30506@boudnik.org>
 <CA+0=VoUzfS6-Z8MJV_KiP3TpnnpK9FF6WqfbD3g_oVkwiQgPsg@mail.gmail.com>
 <CAOeZVieM3SM0v=U8xnQYG_UNBdAuxbMkp6=5MJ9sBc+veXBGfg@mail.gmail.com>
 <CALH+G9oeELA-gsa-w-iAAtLTArNmMPjZE-0daFO7fiN8e=awbw@mail.gmail.com>
 <CAGcMBHg7hsn8qQWhRsBSpzG8=USLVP4TbD_usLEQwYwnp7iWNA@mail.gmail.com>
From: Pavel Tupitsyn <ptupitsyn@gridgain.com>
Date: Tue, 28 Jul 2015 13:15:49 +0300
Message-ID: 
 <CAK2_iMOToPCJ5u9LuOCoDdH5aSu_Weu8OXrP9sqbCW1MvXD18A@mail.gmail.com>
Subject: Re: Important: Git Policy Changed
To: dev@ignite.incubator.apache.org
Content-Type: multipart/alternative; boundary=047d7bd75cb0d8a0f2051becc2f1

--047d7bd75cb0d8a0f2051becc2f1
Content-Type: text/plain; charset=UTF-8

Hi, I'm a bit confused, are we talking about merge vs rebase in a feature
(personal) branches?

On Tue, Jul 28, 2015 at 12:49 PM, Yakov Zhdanov <yzhdanov@apache.org> wrote:

> This can be done from command line.
>
> I think here is a very good article on merge vs rebase -
> https://www.atlassian.com/git/tutorials/merging-vs-rebasing
>
> As for me, I think we should switch to rebase.
>
> --Yakov
>
> 2015-07-28 11:12 GMT+03:00 Alexey Kuznetsov <akuznetsov@gridgain.com>:
>
> > I,m not a git ninja.
> >
> > And will be very pleased if some one add a screenshot to our wiki how to
> do
> > rebase from Idea UI.
> >
> > On Tue, Jul 28, 2015 at 3:06 PM, Atri Sharma <atri.jiit@gmail.com>
> wrote:
> >
> > > +1
> > >
> > > On Tue, Jul 28, 2015 at 1:35 PM, Dmitriy Setrakyan <
> > dsetrakyan@apache.org>
> > > wrote:
> > >
> > > > I actually agree with Cos and think we can switch to "rebase" instead
> > of
> > > > "merge".
> > > >
> > > > Does anyone foresee any problems with this change?
> > > >
> > > > D.
> > > >
> > > > On Mon, Jul 27, 2015 at 2:01 PM, Konstantin Boudnik <cos@apache.org>
> > > > wrote:
> > > >
> > > > > I would like to propose the following modification of the process,
> > > > offered
> > > > > in
> > > > > this page (and it would apply for the ppl working on their branches
> > > > > directly
> > > > > too). Namely:
> > > > >
> > > > > - replace
> > > > >     % git merge master
> > > > >   with
> > > > >     % git rebase master
> > > > >
> > > > > which will allow to avoid empty and non-informative non-fast
> forward
> > > > > commits and also will help to maintain the clean history. You still
> > > need
> > > > to
> > > > > resolve the conflicts though, but most likely you won't have stuff
> > like
> > > > > this sitting around:
> > > > >
> > > > > | *   commit a57707f6ef387ea6f48fd878747e8c5044e7758e
> > > > > | |\  Merge: 0b25ec2 c044bb8
> > > > > | | | Author: Denis Magda <dmagda@gridgain.com>
> > > > > | | | Date:   Wed Jul 22 15:49:04 2015 +0300
> > > > > | | |
> > > > > | | |     Merge branch 'master' of
> > > > > https://git-wip-us.apache.org/repos/asf/incubator-ignite
> > > > > | | |
> > > > > | * | commit 0b25ec2216f7d7c54f29cc9c4ca465c55ed8c0a1
> > > > > | | | Author: Denis Magda <dmagda@gridgain.com>
> > > > > | | | Date:   Wed Jul 22 15:48:36 2015 +0300
> > > > > | | |
> > > > > | | |     muted failing discovery test
> > > > > | | |
> > > > > * | |   commit 2e1dea945b486006214eaa8005a7d3b398c8f549
> > > > > |\ \ \  Merge: 5202527 c044bb8
> > > > > | | |/  Author: S.Vladykin <svladykin@gridgain.com>
> > > > > | |/|   Date:   Wed Jul 22 15:11:38 2015 +0300
> > > > > | | |
> > > > > | | |       Merge branches 'ignite-1015' and 'master' of
> > > > > https://git-wip-us.apache.org/repos/asf/incubator-ignite into
> > > > ignite-1015
> > > > > | | |
> > > > > | * | commit c044bb8278d724336d17fbc2dcd34b0957ac41ec
> > > > > | | | Author: S.Vladykin <svladykin@gridgain.com>
> > > > > | | | Date:   Wed Jul 22 15:10:29 2015 +0300
> > > > > | | |
> > > > > | | |     master - disabled test. fix in ignite-959-z
> > > > > | | |
> > > > > * | |   commit 52025271f3db6e764ab62d0483966159d004b42c
> > > > > |\ \ \  Merge: 301f573 8e1492a
> > > > > | |/ /  Author: S.Vladykin <svladykin@gridgain.com>
> > > > > | | |   Date:   Wed Jul 22 09:04:55 2015 +0300
> > > > > | | |
> > > > > | | |       Merge branch 'master' of
> > > > > https://git-wip-us.apache.org/repos/asf/incubator-ignite into
> > > > ignite-1015
> > > > >
> > > > > - squash the intermediate commits in your own branch, as they are
> > > adding
> > > > no
> > > > >   value to anyone, but polluting the history and making potential
> > > reverts
> > > > >   nearly impossible.
> > > > >
> > > > > With these two changes in the process the patch preparation scripts
> > > > (which
> > > > > I
> > > > > found to be overly complex) would be replaced with a single command
> > > > >
> > > > >     % git format-patch HEAD~1 --stdout > JIRA-###.patch
> > > > >
> > > > > Would be happy to answer any questions, if needed.
> > > > >   Cos
> > > > >
> > > > > On Mon, Jul 27, 2015 at 10:38AM, Dmitriy Setrakyan wrote:
> > > > > > The patch process is also described in "How to Contribute"
> section:
> > > > > >
> > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute
> > > > > >
> > > > > > I think it makes sense to update this page as well.
> > > > > >
> > > > > > D.
> > > > > >
> > > > > > On Mon, Jul 27, 2015 at 10:32 AM, Alexey Goncharuk <
> > > > > > alexey.goncharuk@gmail.com> wrote:
> > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > Alexey, can you provide the link to the updated document? I
> > > cannot
> > > > > find
> > > > > > > it
> > > > > > > > on Wiki or website.
> > > > > > >
> > > > > > >
> > > > > > > It is the document about the sprint process:
> > > > > > >
> > https://cwiki.apache.org/confluence/display/IGNITE/Sprint+Process
> > > > > > >
> > > > > > > --AG
> > > > > > >
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Regards,
> > >
> > > Atri
> > > *l'apprenant*
> > >
> >
> >
> >
> > --
> > Alexey Kuznetsov
> > GridGain Systems
> > www.gridgain.com
> >
>


-- 
-- 
Pavel Tupitsyn
GridGain Systems, Inc.
www.gridgain.com

--047d7bd75cb0d8a0f2051becc2f1--