Sorry for landing late.

Adding separate repo using "git submodule" wn't be good idea here, to get
history/track commits for better maintaince..?

Reference:
http://blog.joncairns.com/2011/10/how-to-use-git-submodules/

On Fri, Apr 27, 2018 at 6:33 AM, Xiaoyu Yao <x...@hortonworks.com> wrote:

> Thanks Owen for fixing this. Sorry, I was not able to spot the issue
> before the merge is pushed up to trunk.
>
> I compared the changes and the results are the same. I also finish ozone
> testing and verify the basic hdfs and ozone functionality are working fine.
> Moving forward, I will add document on Hadoop wiki to avoid similar issue
> in future.
>
> Regards,
> Xiaoyu
>
> On 4/26/18, 9:59 AM, "Chris Douglas" <cdoug...@apache.org> wrote:
>
>     Thank you for doing this, Owen. If the result is the same, then the
>     rebased version is much easier to work with. If not this, then we can
>     squash the branch into a single merge commit. The current state would
>     be a distant, third choice.
>
>     +1 -C
>
>     On Thu, Apr 26, 2018 at 5:49 AM, Owen O'Malley <owen.omal...@gmail.com>
> wrote:
>     > Ok, it was bad enough that I spent yesterday afternoon hacking a fix
> for it.
>     >
>     > I basically did a git rebase with always picking the current master
> version
>     > and then put a new fixup commit to bring the branches into sync.
>     >
>     > The original - https://github.com/omalley/hadoop/tree/hdfs-7240
>     > The rebase hack - https://github.com/omalley/
> hadoop/tree/hdfs-7240-rebase
>     >
>     > Compare the git log --graph difference for one of the recent commits:
>     >
>     > | * | | | | | | | | | | | | | | | | | | | | | | | commit
>     > 919ae746c3a4a424a77e6359af0abe3d7d2c1541
>     > | | | | | | | | | | | | | | | | | | | | | | | | | Author: Weiwei
> Yang <
>     > w...@apache.org>
>     > | | | | | | | | | | | | | | | | | | | | | | | | | Date:   Wed Nov 1
>     > 17:17:03 2017 +0800
>     > | | | | | | | | | | | | | | | | | | | | | | | | |
>     > | | | | | | | | | | | | | | | | | | | | | | | | |     HDFS-12750.
> Ozone:
>     > Fix TestStorageContainerManager#testBlockDeletionTransactions.
> Contributed
>     > by Xiaoyu Yao.
>     > | | | | | | | | | | | | | | | | | | | | | | | | |
>     >
>     > versus
>     >
>     > * commit 055926582222078acef10c3995940510ff09792e
>     > | Author: Weiwei Yang <w...@apache.org>
>     > | Date:   Wed Nov 1 17:17:03 2017 +0800
>     > |
>     > |     HDFS-12750. Ozone: Fix
>     > TestStorageContainerManager#testBlockDeletionTransactions.
> Contributed by
>     > Xiaoyu Yao.
>     > |
>     >
>     > I'd like permission to edit the hadoop git history to make the
> change.
>     >
>     > Thoughts?
>     >
>     > ... Owen
>     >
>     > On Wed, Apr 25, 2018 at 11:47 AM, Xiaoyu Yao <x...@hortonworks.com>
> wrote:
>     >
>     >> We followed the process laid out in a mail thread from 2015 that
> had two
>     >> workflows proposed.
>     >>
>     >> “Git rebase” workflows and “Git merge” workflows.
>     >> Ozone had followed git merge workflow, which merged trunk at a
> regular
>     >> frequency into ozone, and then ozone was merged back.
>     >>
>     >> Here is the mail that we followed for the merge process.
>     >> https://lists.apache.org/thread.html/43cd65c6b6c3c0e8ac
>     >>
>     >> Thanks
>     >> Xiaoyu
>     >>
>     >>
>     >>
>     >>
>     >>
>     >> *From: *Owen O'Malley <owen.omal...@gmail.com>
>     >> *Date: *Wednesday, April 25, 2018 at 11:15 AM
>     >> *To: *Chris Douglas <cdoug...@apache.org>
>     >> *Cc: *Xiaoyu Yao <x...@hortonworks.com>, Jitendra Pandey <
>     >> jiten...@hortonworks.com>, Hdfs-dev <hdfs-dev@hadoop.apache.org>
>     >> *Subject: *Re: [RESULT][VOTE] Merge HDDS (HDFS-7240) *code* into
> trunk
>     >>
>     >>
>     >>
>     >> *sigh*
>     >>
>     >>
>     >>
>     >> I asked you guys to rebase this before merge. It is tempting to
> squash it
>     >> into a single commit.
>     >>
>     >>
>     >>
>     >> .. Owen
>     >>
>     >>
>     >>
>     >> On Wed, Apr 25, 2018 at 11:06 AM, Chris Douglas <
> cdoug...@apache.org>
>     >> wrote:
>     >>
>     >> This really made a mess of trunk. Periodically merging trunk into
> the
>     >> HDFS-7240 branch, then merging the whole thing back, created a
> tangle
>     >> of references that's very difficult to work with (look at the output
>     >> of git log --graph).
>     >>
>     >> I'm not sure it's even possible to fix this, but this is why feature
>     >> branches should rebase and create a merge commit with --no-ff. -C
>     >>
>     >>
>     >>
>     >> On Tue, Apr 24, 2018 at 1:20 PM, Xiaoyu Yao <x...@hortonworks.com>
> wrote:
>     >> > I just merged the branch to trunk (834 commits in total)
>     >> > Again, thanks for all who contributed to HDDS/Ozone!
>     >> >
>     >> > Cheers
>     >> > -Xiaoyu
>     >> >
>     >> > On 4/23/18, 7:26 PM, "Jitendra Pandey" <jiten...@hortonworks.com>
> wrote:
>     >> >
>     >> >     The vote passes with many +1s (12 committers + 5
> contributors) and
>     >> no -1.
>     >> >
>     >> >     Thanks everyone for voting.
>     >> >
>     >> >         On 4/17/18, 5:19 AM, "Jitendra Pandey" <
> jiten...@hortonworks.com>
>     >> wrote:
>     >> >
>     >> >             Hi All,
>     >> >
>     >> >                The community unanimously voted (
> https://s.apache.org/
>     >> HDDSMergeResult) to adopt
>     >> >             HDDS/Ozone as a sub-project of Hadoop, here is the
> formal
>     >> vote for code merge.
>     >> >
>     >> >             Here is a quick summary of the code changes:
>     >> >
>     >> >             - As decided in the JIRA HDFS-10419, the project has
> been
>     >> renamed to Hadoop Distributed Data Store or HDDS.
>     >> >             - HDDS becomes a sub-project of Hadoop.
>     >> >             - Added a Maven profile that disables HDDS
> compilation by
>     >> default.
>     >> >             - The releases of HDDS will be independent of Hadoop
> and
>     >> will have no impact to current Hadoop release process.
>     >> >             - We have made HDDS a loadable module.
>     >> >             - Cleaned up changes in HDFS/Hadoop Common to make
> sure HDDS
>     >> does not impact current users of HDFS.
>     >> >
>     >> >             The vote will run for 7 days, I will start this vote
> with my
>     >> +1.
>     >> >
>     >> >             Thanks
>     >> >             Jitendra
>     >> >
>     >> >
>     >> >             ------------------------------
> -----------------------------
>     >> ----------
>     >> >             To unsubscribe, e-mail: hdfs-dev-unsubscribe@hadoop.
>     >> apache.org
>     >> >             For additional commands, e-mail:
>     >> hdfs-dev-h...@hadoop.apache.org
>     >> >
>     >> >
>     >> >
>     >> >
>     >> >
>     >> >
>     >> >     -----------------------------------------------------------
>     >> ----------
>     >> >     To unsubscribe, e-mail: hdfs-dev-unsubscribe@hadoop.
> apache.org
>     >> >     For additional commands, e-mail: hdfs-dev-help@hadoop.apache.
> org
>     >> >
>     >> >
>     >> >
>     >>
>     >> ------------------------------------------------------------
> ---------
>     >> To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
>     >> For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org
>     >>
>     >>
>     >>
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
> For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org
>
>


-- 



--Brahma Reddy Battula

Reply via email to