Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-29 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/#review186499 --- Ship it! - Madhuvanthi Radhakrishnan On Sept. 27, 2017, 9:52

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/#review186529 --- Ship it! Ship It! - Jayush Luniya On Sept. 27, 2017, 9:52

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Vitalyi Brodetskyi
> On Вер. 27, 2017, 1:28 після полудня, Jayush Luniya wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ANY/scripts/params.py > > Line 246 (original) > > > > > > Lets also fix the Ganglia

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/ --- (Updated Вер. 27, 2017, 9:52 після полудня) Review request for Ambari, Jayush

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/#review186437 --- Ship it! Looks good. Thanks Vitaly! - Madhuvanthi

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Madhuvanthi Radhakrishnan
> On Sept. 27, 2017, 12:20 a.m., Madhuvanthi Radhakrishnan wrote: > > Hi Vitaly, > > Kindly refer https://reviews.apache.org/r/56727/ to check the value of > > user_groups attribute. > > For eg: > > /resources/common-services/ZEPPELIN/0.6.0.2.5/configuration/zeppelin-env.xml > > > >

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Vitalyi Brodetskyi
> On Вер. 27, 2017, 12:20 до полудня, Madhuvanthi Radhakrishnan wrote: > > Hi Vitaly, > > Kindly refer https://reviews.apache.org/r/56727/ to check the value of > > user_groups attribute. > > For eg: > > /resources/common-services/ZEPPELIN/0.6.0.2.5/configuration/zeppelin-env.xml > > > >

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Jayush Luniya
> On Sept. 27, 2017, 12:20 a.m., Madhuvanthi Radhakrishnan wrote: > > Hi Vitaly, > > Kindly refer https://reviews.apache.org/r/56727/ to check the value of > > user_groups attribute. > > For eg: > > /resources/common-services/ZEPPELIN/0.6.0.2.5/configuration/zeppelin-env.xml > > > >

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/#review186408 ---

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-26 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/#review186348 --- Hi Vitaly, Kindly refer https://reviews.apache.org/r/56727/ to

Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-26 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62599/ --- Review request for Ambari, Jayush Luniya and Madhuvanthi Radhakrishnan. Bugs: