Re: [VOTE] Merge feature branch YARN-4752. FairScheduler preemption overhaul

2016-11-23 Thread Karthik Kambatla
Let me conclude the vote with my +1. With 4 +1s and no -1s, the vote passes. Thanks Daniel, Arun and Sangjin for voting. Given the +1 from Jenkins as well, I ll go ahead and merge the branch as a single commit. Thanks Karthik On Thu, Nov 17, 2016 at 10:01 AM, Sangjin Lee

Re: [VOTE] Merge feature branch YARN-4752. FairScheduler preemption overhaul

2016-11-17 Thread Sangjin Lee
+1 On Thu, Nov 17, 2016 at 8:21 AM, Arun Suresh wrote: > +1 > > > On Thu, Nov 17, 2016 at 6:21 AM, Daniel Templeton > wrote: > > > +1 on the merge. Nice work, Karthik! > > > > Daniel > > > > > > On 11/16/16 9:17 PM, Karthik Kambatla wrote: > > > >> Hi

Re: [VOTE] Merge feature branch YARN-4752. FairScheduler preemption overhaul

2016-11-17 Thread Arun Suresh
+1 On Thu, Nov 17, 2016 at 6:21 AM, Daniel Templeton wrote: > +1 on the merge. Nice work, Karthik! > > Daniel > > > On 11/16/16 9:17 PM, Karthik Kambatla wrote: > >> Hi folks, >> >> Based on the positive feedback on the DISCUSS thread [1], I would like to >> start a

Re: [VOTE] Merge feature branch YARN-4752. FairScheduler preemption overhaul

2016-11-17 Thread Daniel Templeton
+1 on the merge. Nice work, Karthik! Daniel On 11/16/16 9:17 PM, Karthik Kambatla wrote: Hi folks, Based on the positive feedback on the DISCUSS thread [1], I would like to start a formal vote to merge feature branch YARN-4752 to trunk. The cumulative patch is posted to YARN-4752, and

[VOTE] Merge feature branch YARN-4752. FairScheduler preemption overhaul

2016-11-16 Thread Karthik Kambatla
Hi folks, Based on the positive feedback on the DISCUSS thread [1], I would like to start a formal vote to merge feature branch YARN-4752 to trunk. The cumulative patch is posted to YARN-4752, and squashed into one commit here [2]. The changes include: 1. Preemption considers individual