Re: Review Request 55482: AM sizing for LLAP - modify YARN min container size to a smaller value leaving all other properties as is

2017-01-13 Thread Sumit Mohanty
> On Jan. 14, 2017, 12:42 a.m., Siddharth Seth wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py, > > line 1044 > > > > > > If we leave minContainerSize unchanged here - the old

Re: Review Request 55482: AM sizing for LLAP - modify YARN min container size to a smaller value leaving all other properties as is

2017-01-13 Thread Sumit Mohanty
> On Jan. 13, 2017, 10:06 p.m., Siddharth Seth wrote: > > Looked at the changes in stack-advisor. Mostly look good to me. > > Was a little confused by the change to minContainerSize based on whether > > the used change the yarn min-allocation-mb property. > > Sumit Mohanty wrote: > So the s

Re: Review Request 55482: AM sizing for LLAP - modify YARN min container size to a smaller value leaving all other properties as is

2017-01-13 Thread Siddharth Seth
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55482/#review161628 --- ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_

Re: Review Request 55482: AM sizing for LLAP - modify YARN min container size to a smaller value leaving all other properties as is

2017-01-13 Thread Siddharth Seth
> On Jan. 13, 2017, 10:06 p.m., Siddharth Seth wrote: > > Looked at the changes in stack-advisor. Mostly look good to me. > > Was a little confused by the change to minContainerSize based on whether > > the used change the yarn min-allocation-mb property. > > Sumit Mohanty wrote: > So the s

Re: Review Request 55482: AM sizing for LLAP - modify YARN min container size to a smaller value leaving all other properties as is

2017-01-13 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55482/#review161602 --- Ship it! Ship It! - Sid Wagle On Jan. 13, 2017, 1:41 a.m., S

Re: Review Request 55482: AM sizing for LLAP - modify YARN min container size to a smaller value leaving all other properties as is

2017-01-13 Thread Sumit Mohanty
> On Jan. 13, 2017, 10:06 p.m., Siddharth Seth wrote: > > Looked at the changes in stack-advisor. Mostly look good to me. > > Was a little confused by the change to minContainerSize based on whether > > the used change the yarn min-allocation-mb property. So the scenarios that I am trying to ca

Re: Review Request 55482: AM sizing for LLAP - modify YARN min container size to a smaller value leaving all other properties as is

2017-01-13 Thread Siddharth Seth
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55482/#review161592 --- Looked at the changes in stack-advisor. Mostly look good to me. Wa

Review Request 55482: AM sizing for LLAP - modify YARN min container size to a smaller value leaving all other properties as is

2017-01-12 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55482/ --- Review request for Ambari, Siddharth Seth, Swapan Shridhar, and Sid Wagle. Repo