Re: Review Request 45566: HAWQ config should not allow multiple Master/Segment directories

2016-04-01 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45566/#review126669 --- Ship it! Ship It! - Matt On April 1, 2016, 2:39 p.m., Lav Ja

Re: Review Request 45566: HAWQ config should not allow multiple Master/Segment directories

2016-04-01 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45566/ --- (Updated April 1, 2016, 9:39 p.m.) Review request for Ambari, Andriy Babiichuk,

Re: Review Request 45566: HAWQ config should not allow multiple Master/Segment directories

2016-04-01 Thread Lav Jain
> On April 1, 2016, 3:58 a.m., Matt wrote: > > ambari-web/app/views/common/configs/service_config_layout_tab_view.js, line > > 56 > > > > > > Can you please confirm if the help text is displayed when you hover > > o

Re: Review Request 45566: HAWQ config should not allow multiple Master/Segment directories

2016-04-01 Thread Lav Jain
> On April 1, 2016, 3:58 a.m., Matt wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py, > > line 972 > > > > > > Is trailing space valid or invalid? > > > > If it is valid,

Re: Review Request 45566: HAWQ config should not allow multiple Master/Segment directories

2016-03-31 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45566/#review126532 --- ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_ad

Review Request 45566: HAWQ config should not allow multiple Master/Segment directories

2016-03-31 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45566/ --- Review request for Ambari, Andriy Babiichuk, Alexander Denissov, Alexandr Antone