Re: Review Request 48490: Last button in the log search pagination panel does not take the user to the last page and few more fixes

2016-06-13 Thread Dharmesh Makwana
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48490/ --- (Updated June 13, 2016, 6:36 a.m.) Review request for Ambari, Alejandro

Re: Review Request 48044: Provide context for hdp-select failures during ambari component install

2016-06-13 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48044/ --- (Updated June 13, 2016, 9:22 a.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48624/#review137264 --- Ship it! Ship It! - Nitiraj Rathore On June 13, 2016, 9:41

Re: Review Request 48623: Fix configuration xml files that don't pass validation

2016-06-13 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48623/#review137253 --- Ship it! Ship It! - Andrew Onischuk On June 13, 2016, 9:24

Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

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

Re: Review Request 48545: View config- Allows remote cluster creation using a user having no permissions on the cluster

2016-06-13 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48545/#review137262 --- Ship it! Ship It! - Nitiraj Rathore On June 10, 2016, 12:15

Review Request 48628: AMBARI-17184: HBase doesn't start because of lacking of variable

2016-06-13 Thread Masahiro Tanaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48628/ --- Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, and Sid Wagle.

Re: Review Request 48485: AMBARI-17044 Optimize LogSearch Solr for cache setting and default values

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

Re: Review Request 48044: Provide context for hdp-select failures during ambari component install

2016-06-13 Thread Laszlo Puskas
> On June 7, 2016, 6:49 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/hdp_select.py, > > line 130 > > > > > > Please consult with Jayush Luniya. > >

Review Request 48622: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-13 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48622/ --- Review request for Ambari, Dharmesh Makwana, Miklos Gergely, Oliver Szabo, and

Re: Review Request 48622: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-13 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48622/#review137260 ---

Re: Review Request 48044: Provide context for hdp-select failures during ambari component install

2016-06-13 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48044/ --- (Updated June 13, 2016, 8:28 a.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 48622: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-13 Thread Don Bosco Durai
> On June 13, 2016, 9:56 a.m., Miklos Gergely wrote: > > ambari-logsearch/ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/output/OutputSolr.java, > > line 354 > > > > > > addRouterField should

Review Request 48623: Fix configuration xml files that don't pass validation

2016-06-13 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48623/ --- Review request for Ambari and Andrew Onischuk. Bugs: AMBARI-17187

Re: Review Request 47941: [AMBARI-16920] Follow up issue for Spark2 stack definition

2016-06-13 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47941/ --- (Updated June 13, 2016, 10:28 a.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 48641: yarncli throws log4j error "FileNotFoundException : /grid/0/log/yarn/hrt_qa/rm-audit.log"

2016-06-13 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48641/#review137305 --- Ship it! Ship It! - Dmytro Sen On Июнь 13, 2016, 2:31 п.п.,

Re: Review Request 48643: Ambari uses too small a window for region server shutdown

2016-06-13 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48643/#review137306 --- Ship it! Ship It! - Dmitro Lisnichenko On June 13, 2016,

Re: Review Request 48642: NPE in ambari-server.out when cluster with kerberos is installed

2016-06-13 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48642/#review137315 --- Ship it! Ship It! - Sebastian Toader On June 13, 2016, 5:04

Re: Review Request 48335: Zeppelin service: Update default zeppelin_pid_dir to /var/run/zeppelin

2016-06-13 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48335/#review137319 --- Ship it! Ship It! - Rohit Choudhary On June 7, 2016, 11:45

Re: Review Request 48650: Add recently added logs to HadoopServiceConfig.json too

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48650/#review137321 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 4:28

Re: Review Request 48642: NPE in ambari-server.out when cluster with kerberos is installed

2016-06-13 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48642/#review137311 --- Fix it, then Ship it! Ship It!

Re: Review Request 48266: Add explicit ambari-server log line indicating cluster creation complete

2016-06-13 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48266/#review137326 --- Thanks for providing this patch. Most of the changes seem

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Andrew Onischuk
> On June 13, 2016, 1:03 p.m., Andrew Onischuk wrote: > > ambari-common/src/main/python/ambari_commons/firewall.py, line 123 > > > > > > Can you please explain why did we remove the check for firewalld? > >

Re: Review Request 48636: Zeppelin Views are not working with Custom and Remote cluster view configuration

2016-06-13 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48636/#review137303 --- Ship it! Ship It! - Rohit Choudhary On June 13, 2016, 1:22

Re: Review Request 48636: Zeppelin Views are not working with Custom and Remote cluster view configuration

2016-06-13 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48636/#review137301 --- Ship it! Ship It! - Gaurav Nagar On June 13, 2016, 1:22

Re: Review Request 48650: Add recently added logs to HadoopServiceConfig.json too

2016-06-13 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48650/#review137329 --- Ship it! Ship It! - Robert Nettleton On June 13, 2016, 4:28

Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread Tom Beerbower
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48655/ --- Review request for Ambari, John Speidel and Nate Cole. Bugs: AMBARI-17203

Review Request 48657: Allow option to skip duplicate URL checking when creating VDF (part 2)

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

Re: Review Request 48657: Allow option to skip duplicate URL checking when creating VDF (part 2)

2016-06-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48657/#review137342 --- Ship it! Ship It! - Jonathan Hurley On June 13, 2016, 1:57

Re: Review Request 48044: Provide context for hdp-select failures during ambari component install

2016-06-13 Thread Alejandro Fernandez
> On June 7, 2016, 6:49 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/hdp_select.py, > > line 130 > > > > > > Please consult with Jayush Luniya. > >

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48624/#review137351 ---

Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

2016-06-13 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48654/ --- Review request for Ambari, Dmytro Sen and Jonathan Hurley. Bugs: AMBARI-17202

Review Request 48659: Fix Spark2 thriftserver Ambari definition bug

2016-06-13 Thread Saisai Shao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48659/ --- Review request for Ambari and Jayush Luniya. Bugs: AMBARI-17204

Re: Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

2016-06-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48654/#review137339 --- Ship it!

Re: Review Request 48657: Allow option to skip duplicate URL checking when creating VDF (part 2)

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

Re: Review Request 48659: Fix Spark2 thriftserver Ambari definition bug

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

Re: Review Request 48557: Fixed implementation of on-ambari-upgrade support. Patch 2: add logic for ambari-upgrade

2016-06-13 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48557/ --- (Updated June 13, 2016, 4:22 p.m.) Review request for Ambari, Alejandro

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Andrew Onischuk
> On June 13, 2016, 1:03 p.m., Andrew Onischuk wrote: > > ambari-common/src/main/python/ambari_commons/firewall.py, line 123 > > > > > > Can you please explain why did we remove the check for firewalld? > >

Re: Review Request 48485: AMBARI-17044 Optimize LogSearch Solr for cache setting and default values

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48485/#review137297 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 10:24

Re: Review Request 48414: AMBARI-17118 Incorrect formated external url in ranger configuration - causes Namenode startup failure

2016-06-13 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48414/ --- (Updated June 13, 2016, 2:15 p.m.) Review request for Ambari, Alejandro

Re: Review Request 48589: Fix HA enabled logic in the alerts

2016-06-13 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48589/ --- (Updated June 13, 2016, 2:21 p.m.) Review request for Ambari, Jonathan Hurley,

Review Request 48634: File browser : File preview show first character only when ssl is enabled.

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

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Masahiro Tanaka
> On June 13, 2016, 1:03 p.m., Andrew Onischuk wrote: > > ambari-common/src/main/python/ambari_commons/firewall.py, line 123 > > > > > > Can you please explain why did we remove the check for firewalld? Thank you

Review Request 48640: Add SERVICE.VIEW_OPERATIONAL_LOGS authorization to SERVICE.ADMINISTRATOR role and above

2016-06-13 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48640/ --- Review request for Ambari, Eugene Chekanskiy, Jonathan Hurley, Nate Cole, and

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48309/#review137283 --- ambari-common/src/main/python/ambari_commons/firewall.py (line

Review Request 48636: Zeppelin Views are not working with Custom and Remote cluster view configuration

2016-06-13 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48636/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Gaurav Nagar,

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Masahiro Tanaka
> On June 13, 2016, 1:53 p.m., Andrew Onischuk wrote: > > Ship It! Thank you! - Masahiro --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48309/#review137293

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Andrew Onischuk
> On June 13, 2016, 1:03 p.m., Andrew Onischuk wrote: > > ambari-common/src/main/python/ambari_commons/firewall.py, line 123 > > > > > > Can you please explain why did we remove the check for firewalld? > >

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48309/#review137293 --- Ship it! Ship It! - Andrew Onischuk On June 7, 2016, 11:11

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Masahiro Tanaka
> On June 13, 2016, 1:03 p.m., Andrew Onischuk wrote: > > ambari-common/src/main/python/ambari_commons/firewall.py, line 123 > > > > > > Can you please explain why did we remove the check for firewalld? > >

Review Request 48642: NPE in ambari-server.out when cluster with kerberos is installed

2016-06-13 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48642/ --- Review request for Ambari, Oliver Szabo, Robert Levas, Sandor Magyari, and

Re: Review Request 48634: File browser : File preview show first character only when ssl is enabled.

2016-06-13 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48634/ --- (Updated June 13, 2016, 1:18 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Masahiro Tanaka
> On June 13, 2016, 1:03 p.m., Andrew Onischuk wrote: > > ambari-common/src/main/python/ambari_commons/firewall.py, line 123 > > > > > > Can you please explain why did we remove the check for firewalld? > >

Re: Review Request 48523: AMBARI-17145 Unformatted configs remain in zeppelin-env.sh

2016-06-13 Thread Masahiro Tanaka
> On June 10, 2016, 9:35 p.m., Jayush Luniya wrote: > > Ship It! Thank you! Could you commit this ? - Masahiro --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48523/#review137073

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Andrew Onischuk
> On June 13, 2016, 1:03 p.m., Andrew Onischuk wrote: > > ambari-common/src/main/python/ambari_commons/firewall.py, line 123 > > > > > > Can you please explain why did we remove the check for firewalld? > >

Re: Review Request 48309: AMBARI-17047: Firewall check returns WARNING even if iptables and firewalld are stopped on CentOS7

2016-06-13 Thread Masahiro Tanaka
> On June 13, 2016, 1:03 p.m., Andrew Onischuk wrote: > > ambari-common/src/main/python/ambari_commons/firewall.py, line 123 > > > > > > Can you please explain why did we remove the check for firewalld? > >

Re: Review Request 48485: AMBARI-17044 Optimize LogSearch Solr for cache setting and default values

2016-06-13 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48485/#review137298 --- Ship it! Ship It! - Miklos Gergely On June 13, 2016, 10:24

Review Request 48641: yarncli throws log4j error "FileNotFoundException : /grid/0/log/yarn/hrt_qa/rm-audit.log"

2016-06-13 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48641/ --- Review request for Ambari and Dmytro Sen. Bugs: AMBARI-17148

Re: Review Request 48634: File browser : File preview show first character only when ssl is enabled.

2016-06-13 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48634/#review137452 --- Ship it! Ship It! - Nitiraj Rathore On June 13, 2016, 1:18

Review Request 48651: Add unit tests for Spark2 service definition

2016-06-13 Thread Saisai Shao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48651/ --- Review request for Ambari and Jayush Luniya. Bugs: AMBARI-16864

Review Request 48650: Add recently added logs to HadoopServiceConfig.json too

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

Re: Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

2016-06-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48654/#review137379 --- Ship it!

Re: Review Request 48266: Add explicit ambari-server log line indicating cluster creation complete

2016-06-13 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48266/#review137384 --- Does this handle UI based cluster deployments? Probably there is

Re: Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

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

Review Request 48643: Ambari uses too small a window for region server shutdown

2016-06-13 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48643/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-16914

Re: Review Request 48642: NPE in ambari-server.out when cluster with kerberos is installed

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48642/#review137308 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 3:04

Re: Review Request 48642: NPE in ambari-server.out when cluster with kerberos is installed

2016-06-13 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48642/#review137312 --- Ship it! Ship It! - Robert Levas On June 13, 2016, 11:04

Review Request 48664: Add logging for the command executed during PXF service check

2016-06-13 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48664/ --- Review request for Ambari, Alexander Denissov and Matt. Bugs: AMBARI-17208

Re: Review Request 48664: Add logging for the command executed during PXF service check

2016-06-13 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48664/#review137415 --- Ship it! Ship It! - Matt On June 13, 2016, 3:30 p.m.,

Review Request 48663: Export PGHOST before any HAWQ Master or Standby custom command is executed

2016-06-13 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48663/ --- Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, and Lav Jain.

Re: Review Request 48663: Export PGHOST before any HAWQ Master or Standby custom command is executed

2016-06-13 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48663/#review137412 --- Ship it! Ship It! - bhuvnesh chaudhary On June 13, 2016,

Re: Review Request 48568: AMBARI-17176 VDF: include default version definition in list, even if Internet Access is available

2016-06-13 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48568/#review137356 --- Ship it! Ship It! - Jaimin Jetly On June 11, 2016, 12:01

Re: Review Request 48557: Fixed implementation of on-ambari-upgrade support. Patch 2: add logic for ambari-upgrade

2016-06-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48557/#review137357 --- Ship it! Ship It! - Nate Cole On June 13, 2016, 9:22 a.m.,

Re: Review Request 48640: Add SERVICE.VIEW_OPERATIONAL_LOGS authorization to SERVICE.ADMINISTRATOR role and above

2016-06-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48640/#review137361 --- Ship it! Ship It! - Jonathan Hurley On June 13, 2016, 2:50

Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread John Speidel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48655/#review137363 --- Ship it! Ship It! - John Speidel On June 13, 2016, 5:51

Re: Review Request 48629: AMBARI-17189 : Change in Atlas authorization from class based to value based

2016-06-13 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48629/#review137364 --- Ship it! Ship It! - Srimanth Gunturi On June 13, 2016,

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Gaurav Nagar
> On June 13, 2016, 6:37 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java, > > line 2440 > > > > > > Move these properties out of the

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

2016-06-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48624/#review137353 ---

Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48655/#review137359 --- Ship it! Ship It! - Nate Cole On June 13, 2016, 1:51 p.m.,

Re: Review Request 48640: Add SERVICE.VIEW_OPERATIONAL_LOGS authorization to SERVICE.ADMINISTRATOR role and above

2016-06-13 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48640/ --- (Updated June 13, 2016, 2:50 p.m.) Review request for Ambari, Eugene

Re: Review Request 48624: Ambari upgrade from 2.2.2 to 2.4.0 fails if cluster name was changed

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

Re: Review Request 48557: Fixed implementation of on-ambari-upgrade support. Patch 2: add logic for ambari-upgrade

2016-06-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48557/#review137360 --- Ship it!

Re: Review Request 48640: Add SERVICE.VIEW_OPERATIONAL_LOGS authorization to SERVICE.ADMINISTRATOR role and above

2016-06-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48640/#review137358 --- Ship it! Ship It! - Nate Cole On June 13, 2016, 2:50 p.m.,

Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48655/#review137373 ---

Re: Review Request 48532: AMBARI-17157 Storm 1.0 log4j config update

2016-06-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48532/#review137375 --- Pushed to trunk, commit bb87fe175aedb7c8eb6400a15b230abfae55da9d

Re: Review Request 48609: AMBARI-17183: client.properties for Falcon should be configurable via Ambari

2016-06-13 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48609/#review137377 --- Has this been tested on an actual stack? These configs are being

Re: Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

2016-06-13 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48654/ --- (Updated Червень 13, 2016, 8:15 після полудня) Review request for Ambari,

Re: Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

2016-06-13 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48654/#review137371 --- Ship it! Ship It! - Dmytro Grinenko On June 13, 2016, 8:15

Re: Review Request 48266: Add explicit ambari-server log line indicating cluster creation complete

2016-06-13 Thread Robert Nettleton
> On June 13, 2016, 4:51 p.m., Robert Nettleton wrote: > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatProcessor.java, > > line 191 > > > > > > I'm a little concerned that this approach

Re: Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

2016-06-13 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48654/ --- (Updated Червень 13, 2016, 8:57 після полудня) Review request for Ambari,

Re: Review Request 47941: [AMBARI-16920] Follow up issue for Spark2 stack definition

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

Re: Review Request 48549: AMBARI-17165 Handle Java patches execution during Ranger upgrade

2016-06-13 Thread Alejandro Fernandez
> On June 10, 2016, 5:40 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.3.xml, > > line 889 > > > > > > hosts="all" is the default, you can remove

Re: Review Request 48308: [AMBARI-17078] Make Spark2-ThriftServer and Livy Server as optional by default

2016-06-13 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48308/ --- (Updated June 14, 2016, 4:10 a.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 48308: [AMBARI-17078] Make Spark2-ThriftServer and Livy Server as optional by default

2016-06-13 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48308/#review137449 --- Ship it! Ship It! - Sumit Mohanty On June 14, 2016, 4:10

Review Request 48670: Return well formatted error response while deleting host with clients installed.

2016-06-13 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48670/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit

Re: Review Request 48308: [AMBARI-17078] Make Spark2-ThriftServer and Livy Server as optional by default

2016-06-13 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48308/#review137445 ---

Re: Review Request 48540: Ubuntu 16, Hive Metastore Start failed

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

Re: Review Request 47941: [AMBARI-16920] Follow up issue for Spark2 stack definition

2016-06-13 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47941/ --- (Updated June 13, 2016, 10:45 a.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 48490: Last button in the log search pagination panel does not take the user to the last page and few more fixes

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48490/#review137275 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 6:36

Re: Review Request 48622: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48622/#review137274 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 9:28

  1   2   >