I want to restart merge vote: Updates: addressed comments from @aw/@chris, fixed:
*Licenses:* - ASF licenses issues - Licenses for dependencies (Added to top level LICENSE.txt). There’s nothing new to add to NOTICE.txt *Building:* - Integrated yarn-ui building to Maven. Instead of build it by default, I placed it under a separate profile (-Pyarn-ui), only people who have interests for this in-development feature can build/try it. - Dockerfile support for dependencies *Documentations:* - Modified BUILDING.txt includes how to build yarn-ui - Moved tutorial doc to site: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/YarnUI2.md *Other changes:* - Removed Unnecessary dot files, including all .gitkeepers - .gitignore is move to top dir - Cleaned up all hard-coded fields for service addresses, such as rm-address / ATS-address, etc. (YARN-4514) Merge JIRA for Jenkins is: <https://issues.apache.org/jira/browse/YARN-4734> https://issues.apache.org/jira/browse/YARN-4734 Latest Jenkins gets back with +1 https://issues.apache.org/jira/browse/YARN-4734?focusedCommentId=15248772&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15248772. (Unit tests failure are not related). (Again :D), the vote will run for 7 days, ending Wednesday, Apr 27. Please feel free to comment if you have any questions/doubts. I'll start with my +1 (binding). + Chris/Allen Thanks, Wangda On Mon, Feb 29, 2016 at 10:50 PM, Wangda Tan <[email protected]> wrote: > Hi Chris, > > Sure, will address comments at YARN-4734 and send a separate DISCUSS > thread before call for vote. > > Thanks, > Wangda > > On Mon, Feb 29, 2016 at 10:40 PM, Allen Wittenauer <[email protected]> > wrote: > >> >> > On Feb 29, 2016, at 9:33 PM, Chris Douglas <[email protected]> wrote: >> > >> > On Thu, Feb 25, 2016 at 8:10 AM, Allen Wittenauer <[email protected]> >> wrote: >> >> -1 (non-binding) >> > >> > Your -1 is binding, per the bylaws. If you intend it only in protest, >> > -0 is typical. >> >> I had it in my head that branch merges requires PMC +1s, not >> committers. It’s definitely not a protest -1. >> >> Thanks. > > >
