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
am"]) ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py <https://reviews.apache.org/r/55482/#comment232886> Think we should retain this rounding functionality, before rounding to the set minContainerSize. That can affect whether we go above or below the minC

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

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
. Was a little confused by the change to minContainerSize based on whether the used change the yarn min-allocation-mb property. - Siddharth Seth On Jan. 13, 2017, 1:41 a.m., Sumit Mohanty wrote: > > --- > This is an automatically ge

Re: Review Request 53801: AMBARI-18901. Use 'Number of LLAP Nodes' selected as the driver for LLAP config calculations.

2016-11-18 Thread Siddharth Seth
- > > (Updated Nov. 18, 2016, 2:34 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Madhuvanthi Radhakrishnan, > Sumit Mohanty, and Siddharth Seth. > > > Bugs: AMBARI-18901 > https://issues.apache.org/jira/browse/AMBARI-18901 > > > Reposi

Re: Review Request 51104: AMBARI-18150. Add Hive2/HiveServer2 config "hive.metastore.event.listeners" as empty value in hive-interactive-site.

2016-08-15 Thread Siddharth Seth
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51104/#review145795 --- Ship it! Ship It! - Siddharth Seth On Aug. 15, 2016, 8:20

Re: Review Request 49014: AMBARI-17331. Determine Tez for Hive2 config 'tez.am.resource.memory.mb' based on cluster capacity.

2016-06-21 Thread Siddharth Seth
it's not set for tez either (and tez will set this to 80% of the container size in this case) - Siddharth Seth On June 21, 2016, 7:59 p.m., Swapan Shridhar wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 49014: AMBARI-17331. Determine Tez for Hive2 config 'tez.am.resource.memory.mb' based on cluster capacity.

2016-06-21 Thread Siddharth Seth
to me. - Siddharth Seth On June 21, 2016, 9:25 a.m., Swapan Shridhar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.a

Re: Review Request 46600: AMBARI-16078. HiveServerInteractive. Config calculations used by LLAP app. and fixes for AMBARI-16079, AMBARI-16086, AMBARI-16087

2016-04-25 Thread Siddharth Seth
resources/stacks/HDP/2.5/services/stack_advisor.py (line 645) <https://reviews.apache.org/r/46600/#comment194130> Eventually, we may need some more changes here based on another config settings - but I think that can be added later. For reference: this is related to using direct memory or a