Re: Review Request 46765: Killing hive metastore and webhcat might fail with "no process" error

2016-04-28 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46765/#review130897 --- Ship it! Ship It! - Laszlo Puskas On April 28, 2016, 9:06

Re: Review Request 46765: Killing hive metastore and webhcat might fail with "no process" error

2016-04-28 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46765/#review130900 --- Ship it! Ship It! - Sandor Magyari On April 28, 2016, 9:06

Re: Review Request 46728: AMBARI-16133: Zeppelin to select only tested Interpreter

2016-04-28 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46728/#review130895 --- Ship it! Ship It! - Pallav Kulshreshtha On April 27, 2016,

Re: Review Request 46769: RU/EU fails due to changes in script.py arguments

2016-04-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46769/#review130902 --- Ship it! Ship It! - Vitalyi Brodetskyi On Квітень 28, 2016,

Review Request 46769: RU/EU fails due to changes in script.py arguments

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

Re: Review Request 46765: Killing hive metastore and webhcat might fail with "no process" error

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

Re: Review Request 46769: RU/EU fails due to changes in script.py arguments

2016-04-28 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46769/ --- (Updated April 28, 2016, 11:14 a.m.) Review request for Ambari and Vitalyi

Re: Review Request 46729: Force running service checks for services before upgrade

2016-04-28 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46729/#review130904 --- Ship it! Ship It! - Nate Cole On April 28, 2016, 3:36 a.m.,

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-28 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46619/ --- (Updated April 28, 2016, 2:03 p.m.) Review request for Ambari, Alejandro

Re: Review Request 46785: Reduce NN start time by removing redundant haadmin calls.

2016-04-28 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46785/ --- (Updated April 28, 2016, 3:22 p.m.) Review request for Ambari and Vitalyi

Re: Review Request 46729: Force running service checks for services before upgrade

2016-04-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46729/#review130919 --- Ship it! Ship It! - Jonathan Hurley On April 28, 2016, 3:36

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-28 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46619/#review130916 --- Ship it! Ship It! - Andrew Onischuk On April 28, 2016, 2:03

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-28 Thread Tim Thorpe
> On April 22, 2016, 11:19 p.m., Matt wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/service_advisor.py, > > line 118 > > > > > > There is some additional logic for HAWQ mentioned in > >

Review Request 46774: JDBC Connect String for Ranger modified wrong for non-default port

2016-04-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46774/ --- Review request for Ambari, Gautam Borad, Mugdha Varadkar, and Velmurugan

Review Request 46776: takeover_config_merge.py should process known template config files

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

Re: Review Request 46773: Sometimes : Warning Alert for Atlas (HTTP 500 response from) is present after upgrade to 2.2.2.0

2016-04-28 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46773/#review130907 --- Ship it! Ship It! - Dmytro Sen On Апрель 28, 2016, 1:06

Re: Review Request 46773: Sometimes : Warning Alert for Atlas (HTTP 500 response from) is present after upgrade to 2.2.2.0

2016-04-28 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46773/ --- (Updated April 28, 2016, 1:06 p.m.) Review request for Ambari, Dmytro Sen and

Re: Review Request 46773: Sometimes : Warning Alert for Atlas (HTTP 500 response from) is present after upgrade to 2.2.2.0

2016-04-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46773/#review130908 --- Ship it! Ship It! - Vitalyi Brodetskyi On Квітень 28, 2016,

Re: Review Request 46751: Ambari metrics API call should allow for early failure

2016-04-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46751/#review130914 --- Ship it! Ship It! - Jonathan Hurley On April 27, 2016, 5:47

Review Request 46797: AMBARI-16082. Test Refresh Phase 1 (#3) (jonathanhurley)

2016-04-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46797/ --- Review request for Ambari, Nate Cole and Sid Wagle. Bugs: AMBARI-16082

Re: Review Request 46797: AMBARI-16082. Test Refresh Phase 1 (#3) (jonathanhurley)

2016-04-28 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46797/#review130996 --- Saw a handful of test errors: Tests in error:

Re: Review Request 45507: Enhance blueprint support for using references

2016-04-28 Thread Shantanu Mundkur
> On April 28, 2016, 9:17 p.m., Shantanu Mundkur wrote: > > Bob, Jayush, > > > > Thanks for reviewing this. > > > > Your review comments for the code changes are valid and Amruta will make > > changes or respond to those. > > > > I have prepared a document that has more details on the

Re: Review Request 46791: Capitalize HAWQ and PXF on service check

2016-04-28 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46791/#review130994 --- Ship it! Ship It! - bhuvnesh chaudhary On April 28, 2016,

Re: Review Request 46800: HAWQ Configuration should be updated whenever Namenode is being moved.

2016-04-28 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46800/#review131000 --- Fix it, then Ship it!

Review Request 46800: HAWQ Configuration should be updated whenever Namenode is being moved.

2016-04-28 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46800/ --- Review request for Ambari, Alexander Denissov, jun aoki, Matt, Oleksandr

Re: Review Request 46776: takeover_config_merge.py should process known template config files

2016-04-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46776/#review130926 --- Ship it! Ship It! - Vitalyi Brodetskyi On Квітень 28, 2016,

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-28 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/ --- (Updated April 28, 2016, 4:34 p.m.) Review request for Ambari, Alexander

Re: Review Request 46746: AMBARI-16141 Add configuration dependencies for Logsearch components

2016-04-28 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46746/ --- (Updated April 28, 2016, 4:48 p.m.) Review request for Ambari, Miklos Gergely,

Review Request 46789: Log_Feeder component logs not picked up

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

Re: Review Request 46789: Log_Feeder component logs not picked up

2016-04-28 Thread Oliver Szabo
> On April 28, 2016, 5:11 p.m., Robert Nettleton wrote: > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/logsearch_common.py, > > line 42 > > > > > > It looks like the

Re: Review Request 46729: Force running service checks for services before upgrade

2016-04-28 Thread Dmitro Lisnichenko
> On April 27, 2016, 8:04 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/CheckDescription.java, > > line 61 > > > > > > Change "rerun" to "run again". Rerun is this guy:

Re: Review Request 46789: Log_Feeder component logs not picked up

2016-04-28 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46789/ --- (Updated April 28, 2016, 5:18 p.m.) Review request for Ambari, Miklos Gergely,

Review Request 46791: Capitalize HAWQ and PXF on service check

2016-04-28 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46791/ --- Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, jun aoki,

Re: Review Request 46789: Log_Feeder component logs not picked up

2016-04-28 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46789/#review130941 --- Fix it, then Ship it! Other than the one issue I've raised,

Re: Review Request 46789: Log_Feeder component logs not picked up

2016-04-28 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46789/#review130950 --- Ship it! Ship It! - Sandor Magyari On April 28, 2016, 5:18

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

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

Re: Review Request 46760: AMBARI-16149. Support for LLAP alert in Ambari

2016-04-28 Thread Swapan Shridhar
> On April 28, 2016, 2:50 p.m., Jonathan Hurley wrote: > > I've never installed LLAP before - Is this data you're getting exposed via > > JMX as a metric? It it was, it would save you a lot of effort in getting > > things like liveInstances. I dont think it's getting exposed via JMX. i'll

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-28 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/#review131009 ---

Re: Review Request 46772: AMBARI-16155. Fixing LLAP calculations and related trivial issues.

2016-04-28 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46772/ --- (Updated April 28, 2016, 11:10 p.m.) Review request for Ambari, Alejandro

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-28 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/#review131002 ---

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-28 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/#review131020 --- Do we have a branch cut with these changes in so that we can test

Re: Review Request 46772: AMBARI-16155. Fixing LLAP calculations and related trivial issues.

2016-04-28 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46772/ --- (Updated April 29, 2016, 12:27 a.m.) Review request for Ambari, Alejandro

Re: Review Request 46692: Improvements to DB consistency check

2016-04-28 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46692/#review131030 --- Ship it! Ship It! - Sumit Mohanty On April 28, 2016, 3:58

Re: Review Request 46772: AMBARI-16155. Fixing LLAP calculations and related trivial issues.

2016-04-28 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46772/#review131031 --- Ship it! Ship It! - Sumit Mohanty On April 29, 2016, 12:40

Re: Review Request 46797: AMBARI-16082. Test Refresh Phase 1 (#3) (jonathanhurley)

2016-04-28 Thread Jonathan Hurley
> On April 28, 2016, 6:20 p.m., Nate Cole wrote: > > Saw a handful of test errors: > > > > > > Tests in error: > > ExecutionCommandWrapperTest.testGetExecutionCommand:213 » NullPointer > > AgentResourceTest.setUp:124 » Creation Guice creation errors: > > > > 1) No implement... > >

Review Request 46808: Role selection in List view of Manage Ambari page does not work correctly

2016-04-28 Thread Keta Patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46808/ --- Review request for Ambari, Di Li and Richard Zang. Bugs: AMBARI-15552

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-04-28 Thread Jeff Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/ --- (Updated April 29, 2016, 3 a.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 46742: Use additional flags for HAWQ Check

2016-04-28 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46742/ --- (Updated April 28, 2016, 6:34 p.m.) Review request for Ambari, Alexander

Re: Review Request 46760: AMBARI-16149. Support for LLAP alert in Ambari

2016-04-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46760/#review130918 --- I've never installed LLAP before - Is this data you're getting

Re: Review Request 46760: AMBARI-16149. Support for LLAP alert in Ambari

2016-04-28 Thread Swapan Shridhar
> On April 28, 2016, 2:50 a.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/alerts/alert_llap_app_status.py, > > line 22 > > > > > > Lets remove all the

Re: Review Request 46729: Force running service checks for services before upgrade

2016-04-28 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46729/ --- (Updated April 28, 2016, 10:36 a.m.) Review request for Ambari, Alejandro

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-28 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/#review130977 --- Ship it! Ship It! - Jayush Luniya On April 28, 2016, 4:36

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-28 Thread Jayush Luniya
> On April 28, 2016, 7:12 p.m., Jayush Luniya wrote: > > Ship It! Thanks @Tim for your contribution. - Jayush --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/#review130977

Re: Review Request 46692: Improvements to DB consistency check

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

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-04-28 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/#review130983 --- Please rebase the patch to the tip of trunk. - Alejandro

Re: Review Request 46760: AMBARI-16149. Support for LLAP alert in Ambari

2016-04-28 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46760/#review130985 ---