Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47949/#review135264 --- Ship it! Ship It! - Alejandro Fernandez On May 27, 2016,

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47949/#review135263 --- Ship it! Ship It! - Sumit Mohanty On May 27, 2016, 4:39

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47949/ --- (Updated May 27, 2016, 4:39 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Swapan Shridhar
> On May 27, 2016, 12:09 p.m., Daniel Gergely wrote: > > > > Daniel Gergely wrote: > Just a quiestion: if \n separated string is needed only for the UI, > wouldnt it be better to handle it on the UI side? So UI can do the > conversion, since it is not needed anywhere else. If it is also

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Daniel Gergely
> On máj. 27, 2016, 12:09 du, Daniel Gergely wrote: > > Just a quiestion: if \n separated string is needed only for the UI, wouldnt it be better to handle it on the UI side? So UI can do the conversion, since it is not needed anywhere else. If it is also used somewhere else, then fine. -

Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47949/ --- Review request for Ambari, Alejandro Fernandez, Daniel Gergely, and Sumit