Review Request 49222: AMBARI-17436: Additional Ambari change for Falcon

2016-06-24 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49222/ --- Review request for Ambari and Alejandro Fernandez. Bugs: AMBARI-17436 https

Re: Review Request 49221: Smart configs for Atlas to setup authentication as either LDAP or AD

2016-06-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49221/#review139423 --- ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/

Re: Review Request 49221: [PREVIEW] Smart configs for Atlas to setup authentication as either LDAP or AD

2016-06-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49221/ --- (Updated June 25, 2016, 12:08 a.m.) Review request for Ambari, Jaimin Jetly, Ab

Review Request 49221: Smart configs for Atlas to setup authentication as either LDAP or AD

2016-06-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49221/ --- Review request for Ambari, Jaimin Jetly, Abhay Kulkarni, Robert Levas, Swapan Sh

Re: Review Request 49220: AMBARI-17434. Keep empty value for Hive2/hive-interactive-site.xml config 'hive.llap.io.memory.mode'.

2016-06-24 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49220/#review139421 --- Ship it! Ship It! - Sumit Mohanty On June 24, 2016, 11:45 p.

Re: Review Request 49220: AMBARI-17434. Keep empty value for Hive2/hive-interactive-site.xml config 'hive.llap.io.memory.mode'.

2016-06-24 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49220/ --- (Updated June 24, 2016, 11:45 p.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 49220: AMBARI-17434. Keep empty value for Hive2/hive-interactive-site.xml config 'hive.llap.io.memory.mode'.

2016-06-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49220/#review139420 --- Ship it! Ship It! - Alejandro Fernandez On June 24, 2016, 11

Review Request 49220: AMBARI-17434. Keep empty value for Hive2/hive-interactive-site.xml config 'hive.llap.io.memory.mode'.

2016-06-24 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49220/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs: AMBARI-

Re: Review Request 49150: AMBARI-17379: set verbose on HostCleanup.py does not work

2016-06-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49150/#review139415 --- Ship it! Ship It! - Alejandro Fernandez On June 23, 2016, 12

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/ --- (Updated June 24, 2016, 8:39 p.m.) Review request for Ambari, Oliver Szabo, Rob

Re: Review Request 49124: AMBARI-17387 Changing HDP repo URL does not leads to sending written URL to ambari-server while registering new HDP version

2016-06-24 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49124/#review139398 --- Ship it! Ship It! - Jaimin Jetly On June 22, 2016, 11:53 p.m

Re: Review Request 49213: Ambari Is Initializing the Metastore Using the Wrong schematool Binary on HDP 2.5

2016-06-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49213/#review139397 --- Ship it! Ship It! - Nate Cole On June 24, 2016, 3:42 p.m., J

Review Request 49213: Ambari Is Initializing the Metastore Using the Wrong schematool Binary on HDP 2.5

2016-06-24 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49213/ --- Review request for Ambari and Nate Cole. Bugs: AMBARI-17431 https://issues.

Re: Review Request 49193: AMBARI-16756 Update code for Ranger Service to use PID file for status and stop call

2016-06-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49193/#review139391 --- Any unit test changes? Did python unit tests pass? - Alejandro Fe

Re: Review Request 49188: Disabling the auto-start for ambari-server and ambari-agent doesn't work on systemd

2016-06-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49188/#review139390 --- ambari-agent/etc/init.d/ambari-agent (line 22)

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/ --- (Updated June 24, 2016, 5:13 p.m.) Review request for Ambari, Oliver Szabo, Rob

Re: Review Request 49204: Incorrect error message for invalid category in Host Combo Search Box

2016-06-24 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49204/#review139385 --- Ship it! Ship It! - Alexandr Antonenko On June 24, 2016, 4:3

Re: Review Request 49173: AMBARI-17416: AMS-Grafana: show 'alias' when there's no datapoint available

2016-06-24 Thread Prajwal Rao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49173/#review139384 --- ambari-metrics/ambari-metrics-grafana/ambari-metrics/datasource.j

Re: Review Request 49205: Fix serviceconfig NPE in db consistency checker

2016-06-24 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49205/#review139380 --- Ship it! Ship It! - Dmytro Sen On Июнь 24, 2016, 4:32 п.п.,

Review Request 49204: Incorrect error message for invalid category in Host Combo Search Box

2016-06-24 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49204/ --- Review request for Ambari, Alexandr Antonenko and Richard Zang. Bugs: AMBARI-17

Review Request 49205: Fix serviceconfig NPE in db consistency checker

2016-06-24 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49205/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Dmytro Sen, Myro

Re: Review Request 49145: Ability to add "javax.jdo.option.ConnectionPassword" to hive clients from ambari

2016-06-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49145/ --- (Updated June 24, 2016, 4:07 p.m.) Review request for Ambari, Robert Levas and

Re: Review Request 49203: Add missing components to LogSearch (part II)

2016-06-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49203/ --- (Updated June 24, 2016, 3:19 p.m.) Review request for Ambari, Oliver Szabo, Rob

Re: Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49195/ --- (Updated June 24, 2016, 3:08 p.m.) Review request for Ambari, Don Bosco Durai,

Re: Review Request 49162: Update Zeppelin service definition

2016-06-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49162/#review139374 --- Ship it! Ship It! - Pallav Kulshreshtha On June 23, 2016, 7:

Re: Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49195/#review139373 --- Ship it! Ship It! - Miklos Gergely On June 24, 2016, 2:14 p.

Re: Review Request 49203: Add missing components to LogSearch (part II)

2016-06-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49203/ --- (Updated June 24, 2016, 2:54 p.m.) Review request for Ambari, Oliver Szabo, Rob

Re: Review Request 49203: Add missing components to LogSearch (part II)

2016-06-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49203/#review139372 --- Ship it! Ship It! - Oliver Szabo On June 24, 2016, 2:48 p.m.

Review Request 49203: Add missing components to LogSearch (part II)

2016-06-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49203/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. B

Re: Review Request 49119: AMBARI-17383: User names should be case insensitive

2016-06-24 Thread Robert Levas
> On June 23, 2016, 3:31 a.m., Myroslav Papirkovskyy wrote: > > ambari-server/src/main/java/org/apache/ambari/server/security/authorization/User.java, > > line 49 > > > > > > IMO user name should be stored and return

Re: Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49195/ --- (Updated June 24, 2016, 2:14 p.m.) Review request for Ambari, Don Bosco Durai,

Re: Review Request 49198: Support Atlas in kerberized environment, submit Service Check as smokeuser

2016-06-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49198/#review139369 --- ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/

Re: Review Request 49198: Support Atlas in kerberized environment, submit Service Check as smokeuser

2016-06-24 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49198/#review139370 --- Ship it! Ship It! - Dmytro Grinenko On June 24, 2016, 1:10 p

Re: Review Request 49199: Falcon client install failed

2016-06-24 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49199/#review139368 --- Ship it! Ship It! - Myroslav Papirkovskyy On Червень 24, 201

Review Request 49199: Falcon client install failed

2016-06-24 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49199/ --- Review request for Ambari, Myroslav Papirkovskyy and Vitalyi Brodetskyi. Bugs:

Review Request 49198: Support Atlas in kerberized environment, submit Service Check as smokeuser

2016-06-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49198/ --- Review request for Ambari, Alejandro Fernandez and Dmytro Grinenko. Bugs: AMBAR

Re: Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49195/ --- (Updated June 24, 2016, 1:05 p.m.) Review request for Ambari, Don Bosco Durai,

Re: Review Request 49197: Ambari server setup failed on debian with postgres local db

2016-06-24 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49197/#review139366 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 24, 2016,

Re: Review Request 49197: Ambari server setup failed on debian with postgres local db

2016-06-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49197/ --- (Updated June 24, 2016, 12:43 p.m.) Review request for Ambari and Vitalyi Brode

Review Request 49197: Ambari server setup failed on debian with postgres local db

2016-06-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49197/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-17426 https:

Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49195/ --- Review request for Ambari, Don Bosco Durai, Miklos Gergely, Robert Nettleton, an

Review Request 49193: AMBARI-16756 Update code for Ranger Service to use PID file for status and stop call

2016-06-24 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49193/ --- Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Gautam Borad, S

Review Request 49194: [Hive view] hiveserver service check fails for LDAP configured hive view

2016-06-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49194/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore, and

Review Request 49188: Disabling the auto-start for ambari-server and ambari-agent doesn't work on systemd

2016-06-24 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49188/ --- Review request for Ambari, Alejandro Fernandez and Andrew Onischuk. Bugs: AMBAR

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-24 Thread Akhil PB
> On June 22, 2016, 10:42 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/capsched.js, > > line 45 > > > > > > Where is isNotOperator used? I guess its

Re: Review Request 48487: Capacity Scheduler View - Node label to queue mapping implementation

2016-06-24 Thread Akhil PB
> On June 22, 2016, 10:12 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components/editQueueCapacity.js, > > line 25 > > > > > > Trailing space must be removed. W