Re: Review Request 46713: AMBARI-16130. Setting 'hive.llap.io.memory.size' config default value to 0.

2016-04-26 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46713/#review130683 --- Ship it! Ship It! - Alejandro Fernandez On April 26, 2016,

Re: Review Request 46475: Provide UI validation for widget_name and description fields in Create/Edit Widget pop-up.

2016-04-26 Thread Keta Patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46475/ --- (Updated April 26, 2016, 9:09 p.m.) Review request for Ambari, Andrii Tkach

Re: Review Request 46714: Prevent Views From Causing a Loss of Service For Ambari

2016-04-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46714/#review130681 ---

Re: Review Request 46714: Prevent Views From Causing a Loss of Service For Ambari

2016-04-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46714/#review130689 --- Ship it!

Re: Review Request 46689: Not able to edit Pig script tab

2016-04-26 Thread Ashwin Rajeev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46689/#review130591 --- Ship it! Ship It! - Ashwin Rajeev On April 26, 2016, 9:55

Review Request 46689: Not able to edit Pig script tab

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

Re: Review Request 46689: Not able to edit Pig script tab

2016-04-26 Thread Gaurav Nagar
> On April 26, 2016, 10:02 a.m., Ashwin Rajeev wrote: > > contrib/views/pig/src/main/java/org/apache/ambari/view/pig/resources/files/FileService.java, > > line 207 > > > > > > Possible NPE if filepath is null, I am

Re: Review Request 46689: Not able to edit Pig script tab

2016-04-26 Thread Ashwin Rajeev
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46689/#review130581 ---

Review Request 46692: Improvements to DB consistency check

2016-04-26 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46692/ --- Review request for Ambari, Dmytro Sen and Sumit Mohanty. Bugs: AMBARI-16094

Review Request 46688: After upgrade ambari-env.sh was changed (WARNING: Can not find ambari-env.sh.rpmsave file from previous version)

2016-04-26 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46688/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-16115

Re: Review Request 46714: Prevent Views From Causing a Loss of Service For Ambari

2016-04-26 Thread Jonathan Hurley
> On April 26, 2016, 7:48 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > lines 555-556 > > > > > > I don't know what kind of views are out

Re: Review Request 46711: 500-error on PUT to the repository_versions endpoint

2016-04-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46711/#review130697 --- Ship it! Ship It! - Jonathan Hurley On April 26, 2016, 4:04

Re: Review Request 46714: Prevent Views From Causing a Loss of Service For Ambari

2016-04-26 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46714/#review130710 --- Ship it! Ship It! - Sid Wagle On April 26, 2016, 10:03

Re: Review Request 46695: User imported from AD is unable to login to Ambari

2016-04-26 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46695/#review130611 ---

Re: Review Request 46619: AMBARI-16085 : Modify Ambari stacks for Ranger (for enabling plugins) to use service keytab for creating repositories and policies

2016-04-26 Thread Gautam Borad
> On April 25, 2016, 5:28 p.m., Nate Cole wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/ranger_functions_v2.py, > > lines 153-155 > > > > > > This is a really long time... the

Re: Review Request 46654: Auth-to-local rule generation duplicates default rules when adding case-insensitive default rules

2016-04-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46654/#review130603 --- Ship it! Ship It! - Jonathan Hurley On April 25, 2016, 3:03

Re: Review Request 46619: AMBARI-16085 : Modify Ambari stacks for Ranger (for enabling plugins) to use service keytab for creating repositories and policies

2016-04-26 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46619/#review130602 ---

Re: Review Request 46698: HBase RegionServers go down after Ambari upgrade due to ClassCastException

2016-04-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46698/ --- (Updated April 26, 2016, 4:56 p.m.) Review request for Ambari, Alejandro

Review Request 46698: HBase RegionServers go down after Ambari upgrade due to ClassCastException

2016-04-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46698/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Mahadev Konar,

Review Request 46695: User imported from AD is unable to login to Ambari

2016-04-26 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46695/ --- Review request for Ambari, Daniel Gergely, Oliver Szabo, and Sandor Magyari.

Re: Review Request 46654: Auth-to-local rule generation duplicates default rules when adding case-insensitive default rules

2016-04-26 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46654/#review130606 --- Ship it! Ship It! - Laszlo Puskas On April 25, 2016, 7:03

Review Request 46697: export_ams_script should save configs into the files

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

Re: Review Request 46619: AMBARI-16085 : Modify Ambari stacks for Ranger (for enabling plugins) to use service keytab for creating repositories and policies

2016-04-26 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46619/ --- (Updated April 26, 2016, 12:18 p.m.) Review request for Ambari, Alejandro

Re: Review Request 46648: Add show_available property to dry_run directive

2016-04-26 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46648/#review130601 --- Ship it! Ship It! - Jonathan Hurley On April 25, 2016, 1:50

Re: Review Request 46684: AMBARI-16112. Removing absolute path for LLAP keytab file (hive.llap.zk.sm.keytab) while creating LLAP slider package.

2016-04-26 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46684/#review130605 --- Ship it! Ship It! - Sumit Mohanty On April 26, 2016, 9:08

Re: Review Request 46688: After upgrade ambari-env.sh was changed (WARNING: Can not find ambari-env.sh.rpmsave file from previous version)

2016-04-26 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46688/#review130604 --- Ship it! Ship It! - Sumit Mohanty On April 26, 2016, 9:36

Re: Review Request 46695: User imported from AD is unable to login to Ambari

2016-04-26 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46695/ --- (Updated April 26, 2016, 4:11 p.m.) Review request for Ambari, Daniel Gergely,

Re: Review Request 46695: User imported from AD is unable to login to Ambari

2016-04-26 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46695/#review130615 --- Ship it! Ship It! - Daniel Gergely On ápr. 26, 2016, 2:17

Re: Review Request 46698: HBase RegionServers go down after Ambari upgrade due to ClassCastException

2016-04-26 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46698/#review130620 --- Ship it! Ship It! - Dmytro Grinenko On April 26, 2016, 1:56

Re: Review Request 46678: Run hadoop haadmin commands with -ns option

2016-04-26 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46678/#review130624 --- Ship it! Ship It! - Dmytro Grinenko On April 26, 2016, 7:56

Re: Review Request 46698: HBase RegionServers go down after Ambari upgrade due to ClassCastException

2016-04-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46698/#review130623 --- Ship it! Ship It! - Nate Cole On April 26, 2016, 11:05

Re: Review Request 46698: HBase RegionServers go down after Ambari upgrade due to ClassCastException

2016-04-26 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46698/#review130625 --- Ship it! Ship It! - Dmytro Grinenko On April 26, 2016, 3:05

Re: Review Request 46698: HBase RegionServers go down after Ambari upgrade due to ClassCastException

2016-04-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46698/ --- (Updated April 26, 2016, 6:05 p.m.) Review request for Ambari, Alejandro

Re: Review Request 46695: User imported from AD is unable to login to Ambari

2016-04-26 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46695/ --- (Updated April 26, 2016, 4:17 p.m.) Review request for Ambari, Daniel Gergely,

Review Request 46703: Reduce error logs on the Sink path if AMS is down

2016-04-26 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46703/ --- Review request for Ambari, Andrew Onischuk, Aravindan Vijayan, and Sid Wagle.

Re: Review Request 46652: Tez View : Equip the tez view shim to work with both version 1 & 2 of Tez UI

2016-04-26 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46652/#review130571 --- Ship it! Ship It! - DIPAYAN BHOWMICK On April 25, 2016,

Re: Review Request 46695: User imported from AD is unable to login to Ambari

2016-04-26 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46695/ --- (Updated April 26, 2016, 6:51 p.m.) Review request for Ambari, Daniel Gergely,

Re: Review Request 46695: User imported from AD is unable to login to Ambari

2016-04-26 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46695/#review130641 --- Ship it! Ship It! - Sandor Magyari On April 26, 2016, 4:51

Re: Review Request 46667: Ambari agent changes to push hive metrics to AMS

2016-04-26 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46667/#review130640 --- AMS changes LGTM. - Aravindan Vijayan On April 26, 2016, 4:08

Re: Review Request 46704: On restarting ambari-agent goes in heartbeat lost state due to other java processes configurations

2016-04-26 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46704/#review130635 --- Ship it! Ship It! - Andrew Onischuk On April 26, 2016, 3:38

Re: Review Request 46702: AMBARI-16122 Logsearch: Property changes not getting reflected at the backend

2016-04-26 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46702/#review130632 --- Ship it! Ship It! - Robert Nettleton On April 26, 2016,

Re: Review Request 46704: On restarting ambari-agent goes in heartbeat lost state due to other java processes configurations

2016-04-26 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46704/#review130628 --- Ship it! Ship It! - Dmytro Sen On Апрель 26, 2016, 3:38

Re: Review Request 46475: Provide UI validation for widget_name and description fields in Create/Edit Widget pop-up.

2016-04-26 Thread Keta Patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46475/ --- (Updated April 26, 2016, 6:45 p.m.) Review request for Ambari, Andrii Tkach

Re: Review Request 46692: Improvements to DB consistency check

2016-04-26 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46692/ --- (Updated Квітень 26, 2016, 6:53 після полудня) Review request for Ambari,

Review Request 46708: Ambari 2.2.1.1 reporting ACID off on slider, but actually on

2016-04-26 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46708/ --- Review request for Ambari, Andrew Onischuk and Dmytro Sen. Bugs: AMBARI-16128

Re: Review Request 46475: Provide UI validation for widget_name and description fields in Create/Edit Widget pop-up.

2016-04-26 Thread Keta Patel
> On April 26, 2016, 5:46 p.m., Andrii Tkach wrote: > > ambari-web/app/controllers/main/service/widgets/create/step3_controller.js, > > line 34 > > > > > > Default value should be boolean, not string. Hello Andrii,

Review Request 46683: Incorrect firewall check on CentOS7 while adding new host to cluster

2016-04-26 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46683/ --- Review request for Ambari, Andrew Onischuk and Vitalyi Brodetskyi. Bugs:

Re: Review Request 46128: AMBARI-15724 Integrate Version Registration in Select Stack Page

2016-04-26 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46128/ --- (Updated April 26, 2016, 8:27 p.m.) Review request for Ambari, Jaimin Jetly,

Re: Review Request 46678: Run hadoop haadmin commands with -ns option

2016-04-26 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46678/ --- (Updated Апрель 26, 2016, 7:56 д.п.) Review request for Ambari, Dmitro

Re: Review Request 46649: Upgrade catalog 220 in not idempotent again

2016-04-26 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46649/#review130559 --- Ship it! Ship It! - Dmitro Lisnichenko On April 25, 2016,

Review Request 46711: 500-error on PUT to the repository_versions endpoint

2016-04-26 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46711/ --- Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. Bugs:

Re: Review Request 46708: Ambari 2.2.1.1 reporting ACID off on slider, but actually on

2016-04-26 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46708/#review130670 --- Ship it! Ship It! - Andrew Onischuk On April 26, 2016, 7:08