[RESULTS] Re: [VOTE] Merge HADOOP-13341

2016-09-12 Thread Allen Wittenauer
The vote passes with 3 +1 binding votes. I'll be merging this later today. Thanks everyone! > On Sep 7, 2016, at 6:44 AM, Allen Wittenauer > wrote: > > > I’d like to call for a vote to run for 5 days (ending Mon 12, 2016 at > 7AM PT) to merge the HADOOP-13341 feature branch into t

Re: [VOTE] Merge HADOOP-13341

2016-09-09 Thread Chris Douglas
+1 (also on JIRA) -C On Wed, Sep 7, 2016 at 6:44 AM, Allen Wittenauer wrote: > > I’d like to call for a vote to run for 5 days (ending Mon 12, 2016 > at 7AM PT) to merge the HADOOP-13341 feature branch into trunk. This branch > was developed exclusively by me. As usual with large shel

Re: [VOTE] Merge HADOOP-13341

2016-09-09 Thread Anu Engineer
> SUBCOMMAND is one of: > > >Clients: > cacheadmin configure the HDFS cache > classpathprints the class path needed to get the hadoop jar > and the required libraries > crypto configure HDFS encryption zones > ... > >Daemons: > bal

Re: [VOTE] Merge HADOOP-13341

2016-09-09 Thread Allen Wittenauer
> On Sep 9, 2016, at 2:15 PM, Anu Engineer wrote: > > +1, Thanks for the effort. It brings in a world of consistency to the hadoop > vars; and as usual reading your bash code was very educative. Thanks! There's still a handful of HDFS and MAPRED vars that begin with HADOOP, b

Re: [VOTE] Merge HADOOP-13341

2016-09-09 Thread Anu Engineer
+1, Thanks for the effort. It brings in a world of consistency to the hadoop vars; and as usual reading your bash code was very educative. I had a minor suggestion though. since we have classified the _OPTS to client and daemon opts, for new people it is hard to know which of these subcommands

Re: [VOTE] Merge HADOOP-13341

2016-09-09 Thread Steve Loughran
> On 8 Sep 2016, at 16:08, Allen Wittenauer wrote: > > >> On Sep 8, 2016, at 2:50 AM, Steve Loughran wrote: >> >> I'm trying to do the review effort here even though I don't know detailed >> bash, as I expect I don't know any less than others, and what better way to >> learn than reviewing

Re: [VOTE] Merge HADOOP-13341

2016-09-08 Thread Allen Wittenauer
> On Sep 8, 2016, at 2:50 AM, Steve Loughran wrote: > > I'm trying to do the review effort here even though I don't know detailed > bash, as I expect I don't know any less than others, and what better way to > learn than reviewing code written by people that do know bash? Just a head

Re: [VOTE] Merge HADOOP-13341

2016-09-08 Thread Steve Loughran
> On 7 Sep 2016, at 14:44, Allen Wittenauer wrote: > > > I’d like to call for a vote to run for 5 days (ending Mon 12, 2016 at > 7AM PT) to merge the HADOOP-13341 feature branch into trunk. This branch was > developed exclusively by me. As usual with large shell script changes, it's

[VOTE] Merge HADOOP-13341

2016-09-07 Thread Allen Wittenauer
I’d like to call for a vote to run for 5 days (ending Mon 12, 2016 at 7AM PT) to merge the HADOOP-13341 feature branch into trunk. This branch was developed exclusively by me. As usual with large shell script changes, it's been broken up into several smaller commits to make it easier