Re: Review Request 60655: AMBARI-21405. Create custom action to force-remove packages

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60655/#review179887 --- Ship it! Ship It! - Jonathan Hurley On July 6, 2017, 2:32

Re: Review Request 60711: Assigning hosts concurrently to same config group may fail with "org.apache.ambari.server.controller.spi.ResourceAlreadyExistsException: Config group already exist'

2017-07-07 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60711/#review179897 --- Ship it! Ship It! - Attila Magyar On July 7, 2017, 5 p.m.,

Re: Review Request 60702: Apply ZEPPELIN-2698 related changes in Ambari

2017-07-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60702/#review179891 ---

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/#review179877 ---

Re: Review Request 60698: AMBARI-21424. BigInsight stack upgrade XML files should prevent downgrade

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60698/#review179886 --- Ship it! Ship It! - Jonathan Hurley On July 6, 2017, 10:20

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60691/#review179888 --- Fix it, then Ship it! I don't know enough about HBase

Review Request 60715: AMBARI-21421 EU upgrade pack to update Hive and Yarn properties to remove IOP/BigSQL properties

2017-07-07 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60715/ --- Review request for Ambari, Dmytro Grinenko, Di Li, Dmitro Lisnichenko, Jonathan

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/ --- (Updated July 7, 2017, 2:31 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 60655: AMBARI-21405. Create custom action to force-remove packages

2017-07-07 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60655/#review179883 --- Ship it! Ship It! - Sid Wagle On July 6, 2017, 6:32 a.m.,

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/#review179892 ---

Re: Review Request 60711: Assigning hosts concurrently to same config group may fail with "org.apache.ambari.server.controller.spi.ResourceAlreadyExistsException: Config group already exist'

2017-07-07 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60711/#review179896 --- Ship it! Ship It! - Robert Levas On July 7, 2017, 1 p.m.,

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/#review179885 --- It has been brought up that this change to blueprints may be too

Review Request 60711: Assigning hosts concurrently to same config group may fail with "org.apache.ambari.server.controller.spi.ResourceAlreadyExistsException: Config group already exist'

2017-07-07 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60711/ --- Review request for Ambari, Attila Magyar, Laszlo Puskas, Robert Levas, and

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60691/#review179890 ---

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Jonathan Hurley
> On July 7, 2017, 1:29 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/script/script.py > > Lines 558 (patched) > > > > > > We should not default to one particular

Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Nate Cole.

Re: Review Request 60695: Add REST end point for the documentation of the Log Feeder shipper properties

2017-07-07 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60695/#review179882 --- Ship it! Ship It! - Robert Nettleton On July 7, 2017, 12:30

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Jonathan Hurley
> On July 7, 2017, 1 p.m., Jonathan Hurley wrote: > > I don't know enough about HBase internals - is removing the write-ahead log > > safe - especially during a complex upgrade? Isn't it like a last line of > > defense in case something goes very wrong? > > Di Li wrote: > yes, this is

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/#review179934 --- Ship it! Ship It! - Alejandro Fernandez On July 7, 2017,

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Alejandro Fernandez
> On July 7, 2017, 5:29 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/script/script.py > > Lines 558 (patched) > > > > > > We should not default to one particular

Re: Review Request 60715: AMBARI-21421 EU upgrade pack to update Hive and Yarn properties to remove IOP/BigSQL properties

2017-07-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60715/#review179937 --- Ship it! Ship It! - Alejandro Fernandez On July 7, 2017,

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Di Li
> On July 7, 2017, 5 p.m., Jonathan Hurley wrote: > > I don't know enough about HBase internals - is removing the write-ahead log > > safe - especially during a complex upgrade? Isn't it like a last line of > > defense in case something goes very wrong? > > Di Li wrote: > yes, this is

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/#review179950 ---

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/ --- (Updated July 7, 2017, 5:48 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60691/ --- (Updated July 7, 2017, 7:34 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60691/#review179939 ---

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Di Li
> On July 7, 2017, 5 p.m., Jonathan Hurley wrote: > > I don't know enough about HBase internals - is removing the write-ahead log > > safe - especially during a complex upgrade? Isn't it like a last line of > > defense in case something goes very wrong? > > Di Li wrote: > yes, this is

Re: Review Request 60702: Apply ZEPPELIN-2698 related changes in Ambari

2017-07-07 Thread Prabhjyot Singh
> On July 7, 2017, 10:52 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/configuration/zeppelin-env.xml > > Line 98 (original), 98 (patched) > > > > > > How

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Di Li
> On July 7, 2017, 5 p.m., Jonathan Hurley wrote: > > I don't know enough about HBase internals - is removing the write-ahead log > > safe - especially during a complex upgrade? Isn't it like a last line of > > defense in case something goes very wrong? yes, this is indeed the last resort as

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/#review179956 --- Ship it! Ship It! - Alejandro Fernandez On July 7, 2017,

Re: Review Request 60709: Allow Multiple Versions of Stack Tools to Co-Exist

2017-07-07 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60709/#review179959 --- Ship it! Ship It! - Sid Wagle On July 7, 2017, 9:48 p.m.,

Re: Review Request 60702: Apply ZEPPELIN-2698 related changes in Ambari

2017-07-07 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60702/#review179855 --- Where is the value exported used in the application? - venkat

Re: Review Request 60689: Upgrade Infra Solr version from 5.5.2 to 6.6.x

2017-07-07 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60689/#review179860 --- Ship it! Ship It! - Miklos Gergely On July 6, 2017, 6:41

Re: Review Request 60695: Add REST end point for the documentation of the Log Feeder shipper properties

2017-07-07 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60695/#review179859 --- Ship it! Ship It! - Oliver Szabo On July 7, 2017, 12:30

Review Request 60707: BE: Extend Ambari REST API to Support User Account Management Improvements

2017-07-07 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60707/ --- Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene Chekanskiy,

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60691/ --- (Updated July 7, 2017, 1:15 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Di Li
> On July 6, 2017, 9:44 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py > > Lines 446 (patched) > > > > > > I believe default()

Re: Review Request 60707: BE: Extend Ambari REST API to Support User Account Management Improvements

2017-07-07 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60707/ --- (Updated July 7, 2017, 9:30 a.m.) Review request for Ambari, Attila Magyar,

Review Request 60708: Log Search UI: implement dynamic loading of options for filters

2017-07-07 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60708/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-21429

Re: Review Request 60708: Log Search UI: implement dynamic loading of options for filters

2017-07-07 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60708/#review179873 --- Ship it! Ship It! - Aleksandr Kovalenko On Июль 7, 2017,

Re: Review Request 60683: Fix execution/custom_commands/custom_actions/service_checks. Send clusterHostInfo

2017-07-07 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60683/ --- (Updated July 7, 2017, 1:54 p.m.) Review request for Ambari, Alejandro

Review Request 60702: Apply ZEPPELIN-2698 related changes in Ambari

2017-07-07 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60702/ --- Review request for Ambari, Attila Doroszlai, Alejandro Fernandez, DIPAYAN