Thanks Sunil!
@Subru and others who voted for force push, are you okay with cancelling this
vote and declaring trunk open for commits?
On 7/6/18, 11:16 AM, "Sunil G" <sun...@apache.org> wrote:
Thanks. These patches are now restored.
- Sunil
On Fri, Jul 6, 2018 at 11:14 AM Vinod Kumar Vavilapalli <vino...@apache.org>
wrote:
> +1
>
> Thanks
> +Vinod
>
>
> On Jul 6, 2018, at 11:12 AM, Sunil G <sun...@apache.org> wrote:
>
> I just checked. YARN-7556 and YARN-7451 can be cherry-picked.
> I cherry-picked in my local and compiled. Things are good.
>
> I can push this now which will restore trunk to its original.
> I can do this if there are no objection.
>
> - Sunil
>
> On Fri, Jul 6, 2018 at 11:10 AM Arpit Agarwal <aagar...@hortonworks.com>
> wrote:
>
> afaict YARN-8435 is still in trunk. YARN-7556 and YARN-7451 are not.
>
>
> From: Giovanni Matteo Fumarola <giovanni.fumar...@gmail.com>
> Date: Friday, July 6, 2018 at 10:59 AM
> To: Vinod Kumar Vavilapalli <vino...@apache.org>
> Cc: Anu Engineer <aengin...@hortonworks.com>, Arpit Agarwal <
> aagar...@hortonworks.com>, "su...@apache.org" <su...@apache.org>, "
> yarn-...@hadoop.apache.org" <yarn-...@hadoop.apache.org>, "
> hdfs-...@hadoop.apache.org" <hdfs-...@hadoop.apache.org>, "
> common-dev@hadoop.apache.org" <common-dev@hadoop.apache.org>, "
> mapreduce-...@hadoop.apache.org" <mapreduce-...@hadoop.apache.org>
> Subject: Re: [VOTE] reset/force push to clean up inadvertent merge commit
> pushed to trunk
>
> Everything seems ok except the 3 commits: YARN-8435, YARN-7556, YARN-7451
> are not anymore in trunk due to the revert.
>
> Haibo/Robert if you can recommit your patches I will commit mine
> subsequently to preserve the original order.
>
> (My apology for the mess I did with the merge commit)
>
> On Fri, Jul 6, 2018 at 10:42 AM, Vinod Kumar Vavilapalli <
> vino...@apache.org<mailto:vino...@apache.org <vino...@apache.org>>> wrote:
> I will add that the branch also successfully compiles.
>
> Let's just move forward as is, unblock commits and just fix things if
> anything is broken.
>
> +Vinod
>
> On Jul 6, 2018, at 10:30 AM, Anu Engineer <aengin...@hortonworks.com
>
> <mailto:aengin...@hortonworks.com <aengin...@hortonworks.com>>> wrote:
>
>
> Hi All,
>
> [ Thanks to Arpit for working offline and verifying that branch is
>
> indeed good.]
>
>
> I want to summarize what I know of this issue and also solicit other
>
> points of view.
>
>
> We reverted the commit(c163d1797) from the branch, as soon as we noticed
>
> it. That is, we have made no other commits after the merge commit.
>
>
> We used the following command to revert
> git revert -c c163d1797ade0f47d35b4a44381b8ef1dfec5b60 -m 1
>
> Giovanni's branch had three commits + merge, The JIRAs he had were
>
> YARN-7451, YARN-7556, YARN-8435.
>
>
> The issue seems to be the revert of merge has some diffs. I am not a
>
> YARN developer, so the only problem is to look at the revert and see if
> there were any spurious edits in Giovanni's original commit + merge.
>
> If there are none, we don't need a reset/force push. But if we find an
>
> issue I am more than willing to go the force commit route.
>
>
> The revert takes the trunk back to the point of the first commit from
>
> Giovanni which is YARN-8435. His branch was also rewriting the order of
> commits which we have lost due to the revert.
>
>
> Based on what I know so far, I am -1 on the force push.
>
> In other words, I am trying to understand why we need the force push. I
>
> have left a similar comment in JIRA (
> https://issues.apache.org/jira/browse/INFRA-16727) too.
>
>
>
> Thanks
> Anu
>
>
> On 7/6/18, 10:24 AM, "Arpit Agarwal" <aagar...@hortonworks.com<mailto:
>
> aagar...@hortonworks.com>> wrote:
>
>
> -1 for the force push. Nothing is broken in trunk. The history looks
>
> ugly for two commits and we can live with it.
>
>
> The revert restored the branch to Giovanni's intent. i.e. only
>
> YARN-8435 is applied. Verified there is no delta between hashes 0d9804d
and
> 39ad989 (HEAD).
>
>
> 39ad989 2018-07-05 aengineer@ o {apache/trunk} Revert "Merge branch
>
> 't...
>
> c163d17 2018-07-05 gifuma@apa M─┐ Merge branch 'trunk' of
>
> https://git-...
>
> 99febe7 2018-07-05 rkanter@ap │ o YARN-7451. Add missing tests to
>
> veri...
>
> 1726247 2018-07-05 haibochen@ │ o YARN-7556. Fair scheduler
>
> configurat...
>
> 0d9804d 2018-07-05 gifuma@apa o │ YARN-8435. Fix NPE when the same
>
> cli...
>
> 71df8c2 2018-07-05 nanda@apac o─┘ HDDS-212. Introduce
>
> NodeStateManager...
>
>
> Regards,
> Arpit
>
>
> On 7/5/18, 2:37 PM, "Subru Krishnan" <su...@apache.org<mailto:
>
> su...@apache.org>> wrote:
>
>
> Folks,
>
> There was a merge commit accidentally pushed to trunk, you can
>
> find the
>
> details in the mail thread [1].
>
> I have raised an INFRA ticket [2] to reset/force push to clean up
>
> trunk.
>
>
> Can we have a quick vote for INFRA sign-off to proceed as this is
>
> blocking
>
> all commits?
>
> Thanks,
> Subru
>
> [1]
>
>
>
http://mail-archives.apache.org/mod_mbox/hadoop-yarn-dev/201807.mbox/%3CCAHqguubKBqwfUMwhtJuSD7X1Bgfro_P6FV%2BhhFhMMYRaxFsF9Q%40mail.gmail.com%3E
>
> [2] https://issues.apache.org/jira/browse/INFRA-16727
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: common-dev-unsubscr...@hadoop.apache.org
>
> <mailto:common-dev-unsubscr...@hadoop.apache.org
> <common-dev-unsubscr...@hadoop.apache.org>>
>
> For additional commands, e-mail: common-dev-h...@hadoop.apache.org
>
> <mailto:common-dev-h...@hadoop.apache.org
> <common-dev-h...@hadoop.apache.org>>
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: common-dev-unsubscr...@hadoop.apache.org<mailto:
>
> common-dev-unsubscr...@hadoop.apache.org>
>
> For additional commands, e-mail: common-dev-h...@hadoop.apache.org
>
> <mailto:common-dev-h...@hadoop.apache.org
> <common-dev-h...@hadoop.apache.org>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: yarn-dev-unsubscr...@hadoop.apache.org<mailto:
> yarn-dev-unsubscr...@hadoop.apache.org>
> For additional commands, e-mail: yarn-dev-h...@hadoop.apache.org<mailto:
> yarn-dev-h...@hadoop.apache.org>
>
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: common-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-dev-h...@hadoop.apache.org
