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> 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> 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 For additional commands, e-mail: common-dev-h...@hadoop.apache.org
