Re: Review Request 58480: Files View: In non-HA environment, Files view executes with error when "List of NameNodes" property is cleared

2017-04-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58480/#review172101 --- Ship it! Ship It! - Di Li On April 17, 2017, 4:51 p.m

Review Request 58534: AMBARI-20784 Able to hide the "Why public repo not selected" when no stacks have repoinfo.xml with the "latest" section

2017-04-19 Thread Di Li
rify the "Why not selected" is hidden. and when some stacks have the section, the link shows up if no stacks have repo with stackDefault=False. Thanks, Di Li

Re: Review Request 58538: AMBARI-20780 - Update HBase Configuration group is ignored in upgrade.xml

2017-04-19 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58538/#review172378 --- Ship it! Ship It! - Di Li On April 19, 2017, 5:59 p.m., Tim

Re: Review Request 58591: Roles page in Ambari UI shows extra rows in IE browser

2017-04-21 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58591/#review172646 --- Ship it! Ship It! - Di Li On April 20, 2017, 8:34 p.m

Review Request 58665: AMBARI-20822 Ambari Agent needs to handle custom version number like 1.2.3.4_Company in its Controller code

2017-04-24 Thread Di Li
tests, new unit test, patched a cluster with the fix and verify the error message message was gone. Thanks, Di Li

Re: Review Request 58671: check_ambari_permissions.py does not run for all the files and directories listed

2017-04-24 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58671/#review172816 --- Ship it! Ship It! - Di Li On April 24, 2017, 4:36 p.m

Re: Review Request 58665: AMBARI-20822 Ambari Agent needs to handle custom version number like 1.2.3.4_Company in its Controller code

2017-04-24 Thread Di Li
rg/r/58665/diff/2/ Changes: https://reviews.apache.org/r/58665/diff/1-2/ Testing --- exiting unit tests, new unit test, patched a cluster with the fix and verify the error message message was gone. Thanks, Di Li

Re: Review Request 58590: Provide an optional attribute for custom commands to specify title used in operations window title

2017-04-25 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58590/#review172913 --- Ship it! Ship It! - Di Li On April 25, 2017, 4:51 a.m

Re: Review Request 58715: WFM: Date filter in workflow dashboard is not working properly

2017-04-25 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58715/#review172961 --- Ship it! Ship It! - Di Li On April 25, 2017, 7:06 p.m

Re: Review Request 58845: AMBARI-20891 - Allow extensions to auto-link with supported stack versions

2017-04-28 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58845/#review173336 --- Ship it! Ship It! - Di Li On April 28, 2017, 3:10 p.m., Tim

Re: Review Request 58429: After pam setup- Hive View user home test fails

2017-04-28 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58429/#review173367 --- Ship it! Ship It! - Di Li On April 28, 2017, 5:45 p.m

Review Request 58919: AMBARI-20903 RepoVersion parsing error should be more specific

2017-05-02 Thread Di Li
/server/orm/entities/RepositoryVersionEntity.java d3705f3 Diff: https://reviews.apache.org/r/58919/diff/1/ Testing --- existing unit tests. patch a cluster with the change. Thanks, Di Li

Re: Review Request 58927: AMBARI-20908 Kafka advertised.listeners replaced in Kerberos Mode

2017-05-02 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58927/#review173612 --- Ship it! Ship It! - Di Li On May 2, 2017, 5:28 p.m., Bharat

Re: Review Request 58968: Server Error in Ambari UI, when trying to login as a pam user due to user name conflict

2017-05-03 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58968/#review173746 --- Ship it! Ship It! - Di Li On May 3, 2017, 4:48 p.m., Anita

Re: Review Request 58864: AMBARI-20698 Ability to export blueprint via Ambari installer UI

2017-05-03 Thread Di Li
On May 1, 2017, 9:06 p.m., Amruta Borkar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58864/ > --- > >

Re: Review Request 58864: AMBARI-20698 Ability to export blueprint via Ambari installer UI

2017-05-03 Thread Di Li
"Generate Blueprint" or something hinting the files will be generated ? - Di Li On May 1, 2017, 9:06 p.m., Amruta Borkar wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 58894: WFM: Include an option to clear filters in workflow dashboard

2017-05-04 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58894/#review173941 --- Ship it! Ship It! - Di Li On May 1, 2017, 10:15 p.m., Anita

Re: Review Request 58864: AMBARI-20698 Ability to export blueprint via Ambari installer UI

2017-05-05 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58864/#review174044 --- Ship it! Ship It! - Di Li On May 5, 2017, 12:21 a.m

Re: Review Request 58845: AMBARI-20891 - Allow extensions to auto-link with supported stack versions

2017-05-08 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58845/#review174193 --- Ship it! Ship It! - Di Li On May 5, 2017, 6:30 p.m., Tim

Review Request 59232: AMBARI-20984 Be able to include stack repos in blueprint

2017-05-12 Thread Di Li
"override_strategy":"EXISTING", "operating_system":"redhat6", "repo_id": "HDP-UTILS-1.1.0.21", "repo_name": "HDP-UTILS", "base_url": "http://public-repo-1.hortonworks.com/HDP-UTILS-1.1.0.21/repos/centos7"; } ] } ], Thanks, Di Li

Re: Review Request 59232: AMBARI-20984 Be able to include stack repos in blueprint

2017-05-12 Thread Di Li
PLY_WHEN_MISSING", "operating_system":"redhat7", "repo_id": "HDP-UTILS-1.1.0.21", "base_url": "http://public-repo-1.hortonworks.com/HDP-UTILS-1.1.0.21/repos/centos7"; } ] } ], Thanks, Di Li

Re: Review Request 59232: AMBARI-20984 Be able to include stack repos in blueprint

2017-05-12 Thread Di Li
"override_strategy":"APPLY_WHEN_MISSING", "operating_system":"redhat7", "repo_id": "HDP-UTILS-1.1.0.21", "base_url": "http://public-repo-1.hortonworks.com/HDP-UTILS-1.1.0.21/repos/centos7"; } ] } ], Thanks, Di Li

Re: Review Request 59232: AMBARI-20984 Be able to include stack repos in blueprint

2017-05-12 Thread Di Li
"override_strategy":"APPLY_WHEN_MISSING", "operating_system":"redhat7", "repo_id": "HDP-UTILS-1.1.0.21", "base_url": "http://public-repo-1.hortonworks.com/HDP-UTILS-1.1.0.21/repos/centos7"; } ] } ], Thanks, Di Li

Re: Review Request 59373: Address config-groups while exporting blueprint from UI

2017-05-19 Thread Di Li
(original), 2343 (patched) <https://reviews.apache.org/r/59373/#comment248938> minor: remove console.log from tests - Di Li On May 18, 2017, 6:27 p.m., Amruta Borkar wrote: > > --- > This is an automatically generated e

Re: Review Request 59382: Regex pattern for version number in Register Version UI should depend on stack's version dir

2017-05-19 Thread Di Li
/services/Stack.js Lines 150 (patched) <https://reviews.apache.org/r/59382/#comment248939> nitpicking: remove console.log line. - Di Li On May 19, 2017, 5:23 a.m., Sangeeta Ravindran wrote: > > --- > This is an automatically g

Re: Review Request 59151: Workflow Designer View: Provide sorting of workflows on dashboard

2017-05-19 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59151/#review175504 --- Ship it! Ship It! - Di Li On May 10, 2017, 6:33 p.m., Anita

Re: Review Request 59382: Regex pattern for version number in Register Version UI should depend on stack's version dir

2017-05-23 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59382/#review175793 --- Ship it! Ship It! - Di Li On May 19, 2017, 9:16 p.m

Re: Review Request 59373: Address config-groups while exporting blueprint from UI

2017-05-23 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59373/#review175797 --- Ship it! Ship It! - Di Li On May 19, 2017, 5:01 p.m

Re: Review Request 59232: AMBARI-20984 Be able to include stack repos in blueprint

2017-05-23 Thread Di Li
uot;; }, { "override_strategy":"APPLY_WHEN_MISSING", "operating_system":"redhat7", "repo_id": "HDP-UTILS-1.1.0.21", "base_url": "http://public-repo-1.hortonworks.com/HDP-UTILS-1.1.0.21/repos/centos7"; } ] } ], Thanks, Di Li

Re: Review Request 59532: Add Confirmation pop up for 'Generate Blueprint' button

2017-05-24 Thread Di Li
ng the cluster. - Di Li On May 24, 2017, 7:30 p.m., Amruta Borkar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 59532: Add Confirmation pop up for 'Generate Blueprint' button

2017-05-25 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59532/#review176064 --- Ship it! Ship It! - Di Li On May 24, 2017, 10:43 p.m

Re: Review Request 59609: Export cluster_name in the cluster template file

2017-05-31 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59609/#review176459 --- Ship it! Ship It! - Di Li On May 30, 2017, 10:08 p.m

Re: Review Request 59649: Delete view privileges from the Users page

2017-06-01 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59649/#review176633 --- Ship it! Ship It! - Di Li On May 30, 2017, 8:10 p.m., Anita

Re: Review Request 59609: Export cluster_name in the cluster template file

2017-06-01 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59609/#review176649 --- please close this one if it has been committed - Di Li On May

Re: Review Request 59649: Delete view privileges from the Users page

2017-06-01 Thread Di Li
> On June 1, 2017, 5:39 p.m., Di Li wrote: > > Ship It! > > Anita Jebaraj wrote: > Can you please help in pushing the changes? done, you can close this request now - Di --- This is an automatically generate

Review Request 59727: AMBARI-21161 Able to safe guard Ambari server against circular dependencies in role command order

2017-06-01 Thread Di Li
message when try to start HDFS service, and does not really kick off the HDFS start. (without the fix, Ambari server will try to start HDFS and gets itself into an infinite loop and uses up 100% CPU and large amount of RAM) Thanks, Di Li

Re: Review Request 59727: AMBARI-21161 Able to safe guard Ambari server against circular dependencies in role command order

2017-06-01 Thread Di Li
depends on NN start. Verify UI shows an error message when try to start HDFS service, and does not really kick off the HDFS start. (without the fix, Ambari server will try to start HDFS and gets itself into an infinite loop and uses up 100% CPU and large amount of RAM) Thanks, Di Li

Re: Review Request 59639: Blueprint export should allow tokenized values in SingleHostUpdater

2017-06-05 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59639/#review176917 --- Ship it! Ship It! - Di Li On June 1, 2017, 9:58 p.m

Re: Review Request 59727: AMBARI-21161 Able to safe guard Ambari server against circular dependencies in role command order

2017-06-06 Thread Di Li
server will try to start HDFS and gets itself into an infinite loop and uses up 100% CPU and large amount of RAM) Thanks, Di Li

Re: Review Request 59727: AMBARI-21161 Able to safe guard Ambari server against circular dependencies in role command order

2017-06-06 Thread Di Li
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59727/#review176659 --- On June 1, 2017, 8:35 p.m., Di Li wrote: > > -

Re: Review Request 59838: Stack advisor needs to recommend dependency for slaves and masters

2017-06-06 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59838/#review177078 --- Ship it! Ship It! - Di Li On June 6, 2017, 11:33 a.m., Tim

Re: Review Request 59727: AMBARI-21161 Able to safe guard Ambari server against circular dependencies in role command order

2017-06-06 Thread Di Li
gt; Btw, this will only find cycles between two adjacent nodes. It will not > > find cycles involved with 3 nodes, e.g., > > > > A -> B > > B -> C > > C -> A > > > > This needs a 2-pass depth-first traversal or Tarjan

Re: Review Request 59727: AMBARI-21161 Able to safe guard Ambari server against circular dependencies in role command order

2017-06-06 Thread Di Li
server will try to start HDFS and gets itself into an infinite loop and uses up 100% CPU and large amount of RAM) Thanks, Di Li

Re: Review Request 59838: Stack advisor needs to recommend dependency for slaves and masters

2017-06-07 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59838/#review177170 --- Ship it! Ship It! - Di Li On June 7, 2017, 12:26 p.m., Tim

Review Request 59913: AMBARI-21197 Disable Kerberos UI step by step panel misalign with the Complete button at the bottom of the page

2017-06-08 Thread Di Li
kerberos, verify the progress page now has the correct width (via the new if/else logic) Thanks, Di Li

Re: Review Request 59919: YARN service advisor has spelling error in yarn-site

2017-06-08 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59919/#review177315 --- Ship it! Ship It! - Di Li On June 8, 2017, 3:16 p.m., Tim

Review Request 59953: AMBARI-21213 Consolidate UI for the "Loading..." text

2017-06-09 Thread Di Li
ambari-web/test/views/main/alerts/definition_details_view_test.js b3c4712 Diff: https://reviews.apache.org/r/59953/diff/1/ Testing --- unit tests. patch trunk cluster with changes. Thanks, Di Li

Review Request 59955: AMBARI-21218 Enable Kerberos UI is blank the first time opens it

2017-06-09 Thread Di Li
955/diff/1/ Testing --- unit tests. patch a trunk cluster and verify the issue is fixed. Thanks, Di Li

Re: Review Request 60011: Remove Zookeeper as a required service from YARN

2017-06-12 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60011/#review177664 --- Ship it! Ship It! - Di Li On June 12, 2017, 5:44 p.m., Tim

Re: Review Request 59838: Stack advisor needs to recommend dependency for slaves and masters

2017-06-19 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59838/#review178255 --- Ship it! Ship It! - Di Li On June 13, 2017, 9:56 p.m., Tim

Review Request 60206: AMBARI-21281 Code refactoring: rename RU_TASKS Stagewrapper type to upgrade tasks

2017-06-19 Thread Di Li
tests. Thanks, Di Li

Review Request 60390: AMBARI-21327 Ambari server to print error messages if NN HA namenode services properties use diff FQDN (dual network cards) than FQDN in the HostComponentState table

2017-06-23 Thread Di Li
e/ambari/server/stack/MasterHostResolver.java 427a5f5 Diff: https://reviews.apache.org/r/60390/diff/1/ Testing --- existing unit tests. Thanks, Di Li

Re: Review Request 60477: AMBARI-21350. Create a cross stack upgrade pack in Ambari

2017-06-27 Thread Di Li
/config-upgrade.xml Lines 1 (patched) <https://reviews.apache.org/r/60477/#comment253420> Should we even allow the auto config updates during EU ? Tim and I tried the EU with just the nonrolling-upgrade.xml that we copied over from HDP 2.6 stack. - Di Li On June 27, 2017, 6:24 p.m., Ale

Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

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

Review Request 60481: AMBARI-21359 Add JavaVersionCheck check

2017-06-27 Thread Di Li
/java/org/apache/ambari/server/checks/CheckDescription.java 71c5857 ambari-server/src/main/java/org/apache/ambari/server/checks/JavaVersionCheck.java PRE-CREATION Diff: https://reviews.apache.org/r/60481/diff/1/ Testing --- to be tested later on with the integrated tests. Thanks, Di

Re: Review Request 60489: Typo in stack advisor error message for yarn and mr queue config issues

2017-06-28 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60489/#review179097 --- Ship it! Ship It! - Di Li On June 28, 2017, 5:23 a.m

Re: Review Request 60488: Ability to delete a view instance from view instance list

2017-06-28 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60488/#review179098 --- Ship it! Ship It! - Di Li On June 28, 2017, 5:21 a.m

Re: Review Request 60481: AMBARI-21359 Add JavaVersionCheck check

2017-06-28 Thread Di Li
://reviews.apache.org/r/60481/diff/1-2/ Testing --- to be tested later on with the integrated tests. Thanks, Di Li

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179109 --- On June 27, 2017, 8:08 p.m., Di Li wrote: > > --- > This is an automatically gen

Re: Review Request 60481: AMBARI-21359 Add JavaVersionCheck check

2017-06-28 Thread Di Li
://reviews.apache.org/r/60481/diff/2-3/ Testing --- to be tested later on with the integrated tests. Thanks, Di Li

Re: Review Request 60481: AMBARI-21359 Add JavaVersionCheck check

2017-06-28 Thread Di Li
--------- On June 28, 2017, 3:16 p.m., Di Li wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60481/ > -

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. Bugs: AMBARI-21351 https://issues.apache.org/jira/browse/AMBARI-21351 Repository: ambari Description --- Some stacks, such as BigInights, support MariaDB as the database for Hive. When upgrading

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
> On June 28, 2017, 2:48 p.m., Jonathan Hurley wrote: > > Should this also be put into trunk? We would still need to support this > > moving forward. Maybe we need a Jira to track an actual property which > > governs which database executable to invoke. > > Di

Review Request 60509: AMBARI-21368 Update Ambari server rat exclude list with BI AMS files

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

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179142 ------- On June 28, 2017, 6:23 p.m., Di Li wrote: > > -

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. Bugs: AMBARI-21351 https://issues.apache.org/jira/browse/AMBARI-21351 Repository: ambari Description --- Some stacks, such as BigInights, support MariaDB as the database for Hive. When upgrading

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179157 ------- On June 28, 2017, 6:23 p.m., Di Li wrote: > > ---

Review Request 60537: AMBARI-21373 Remove clean repo logic not suitable for Apache Ambari Python API

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

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-29 Thread Di Li
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179157 --- On June 28, 2017, 6:23 p.m., Di Li wrote: > > -

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-29 Thread Di Li
, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. Bugs: AMBARI-21351 https://issues.apache.org/jira/browse/AMBARI-21351 Repository: ambari Description --- Some stacks, such as BigInights, support MariaDB as the database for Hive. When upgrading

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-29 Thread Di Li
nfigs and have Hive report > staleness and require a restart. Instead, it seems like we should default to > False if not found and use the upgrade packs to update this. > > In other words; since Stack-A might need this when moving to Stack-B, > then it should provide this as

Review Request 60574: AMBARI-21383 Update upgrade pack to set mariadb support property for hive

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

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-07-06 Thread Di Li
, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. Bugs: AMBARI-21351 https://issues.apache.org/jira/browse/AMBARI-21351 Repository: ambari Description --- Some stacks, such as BigInights, support MariaDB as the database for Hive. When upgrading

Review Request 60690: AMBARI-21419 EU to auto switch to HDP tomcat for Oozie

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

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

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

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
, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. Bugs: AMBARI-21420 https://issues.apache.org/jira/browse/AMBARI-21420 Repository: ambari Description --- HBase in IOP 4.2.5 is of higher version than the one shipped in HDP 2.6. HBase master

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/#review179830 --- On July 6, 2017, 9:08 p.m., Di Li wrote: > > -

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 60691: AMBARI-21420 HBase master crashed during/post EU on an IOP4.2.5/HDP2.6 migrated cluster

2017-07-07 Thread Di Li
, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. Bugs: AMBARI-21420 https://issues.apache.org/jira/browse/AMBARI-21420 Repository: ambari Description --- HBase in IOP 4.2.5 is of higher version than the one shipped in HDP 2.6. HBase master

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? >

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
tests showed that it worked on a secured cluster, dropping the issue for now. - Di --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60691/#review179888 -------

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? >

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

2017-07-10 Thread Di Li
, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. Bugs: AMBARI-21420 https://issues.apache.org/jira/browse/AMBARI-21420 Repository: ambari Description --- HBase in IOP 4.2.5 is of higher version than the one shipped in HDP 2.6. HBase master

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

2017-07-10 Thread Di Li
, Di Li, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty, Sid Wagle, and Tim Thorpe. Changes --- backup the old wal dir instead of removing it. Bugs: AMBARI-21420 https://issues.apache.org/jira/browse/AMBARI-21420 Repository: ambari Description --- HBase in IOP 4.2.5 is of

Review Request 60754: AMBARI-21439 Hive/MapRed2 restart fails after Ambari upgrade

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

Review Request 60807: AMBARI-21453 Restart Hive after Install Packages fails with err on Condition with name 'should_install_mysl_connector'

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

Review Request 60814: AMBARI-21461 Spark thrift server restart fails during migration EU

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

Re: Review Request 60814: AMBARI-21461 Spark thrift server restart fails during migration EU

2017-07-13 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60814/#review180361 ------- On July 12, 2017, 9 p.m., Di Li wrote: > > --

Review Request 60845: AMBARI-21468 Knox restart fails during EU due to wrong location to the Knox data directory

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

Re: Review Request 60845: AMBARI-21468 Knox restart fails during EU due to wrong location to the Knox data directory

2017-07-13 Thread Di Li
. Let me know if this can be done differently (better ?) - Di Li On July 13, 2017, 4:27 p.m., Di Li wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 60862: AMBARI-21474. HBase REST Server is missing after Ambari upgrade

2017-07-14 Thread Di Li
before Ambari upgrade. Shouldn't that resolve this issue ? 2. BigInsights 4.2.5 HBase also has its own configurations. Should those be ported as well ? - Di Li On July 14, 2017, 12:23 p.m., Attila Doroszlai wrote: > > --- &

Review Request 60866: AMBARI-21477: Remove Falcon proxy entries from Knox kerberos.json

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

Re: Review Request 60862: AMBARI-21474. HBase REST Server is missing after Ambari upgrade

2017-07-14 Thread Di Li
> On July 14, 2017, 1:13 p.m., Di Li wrote: > > 1. I thought the agreement was to remove IBM services/components before > > Ambari upgrade. Shouldn't that resolve this issue ? > > 2. BigInsights 4.2.5 HBase also has its own configurations. Should those be > >

Re: Review Request 60862: AMBARI-21474. HBase REST Server is missing after Ambari upgrade

2017-07-14 Thread Di Li
> On July 14, 2017, 1:13 p.m., Di Li wrote: > > 1. I thought the agreement was to remove IBM services/components before > > Ambari upgrade. Shouldn't that resolve this issue ? > > 2. BigInsights 4.2.5 HBase also has its own configurations. Should those be > >

Re: Review Request 60986: AMBARI-21528. Zookeeper server has incorrect memory setting, missing m in Xmx value

2017-07-20 Thread Di Li
86/#review181042 --- On July 20, 2017, 12:13 a.m., Alejandro Fernandez wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60986/ > --

Review Request 61000: AMBARI-21527 Restart of MR2 History Server failed due to wrong NameNode RPC address

2017-07-20 Thread Di Li
r/src/main/resources/stacks/BigInsights/4.2/upgrades/config-upgrade.xml dada6e2 ambari-server/src/main/resources/stacks/BigInsights/4.2/upgrades/nonrolling-upgrade-to-hdp-2.6.xml 3ea20ed Diff: https://reviews.apache.org/r/61000/diff/1/ Testing --- tested as part of the EU Thanks, Di Li

Re: Review Request 61000: AMBARI-21527 Restart of MR2 History Server failed due to wrong NameNode RPC address

2017-07-20 Thread Di Li
---------- On July 20, 2017, 4:54 p.m., Di Li wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61000/ >

Review Request 61006: AMBARI-21541 Restart services failed post Ambari Upgrade

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

Re: Review Request 61012: AMBARI-21463. Cross-stack upgrade, Oozie restart fails with ext-2.2.zip missing error

2017-07-21 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61012/#review181113 --- Ship it! Ship It! - Di Li On July 20, 2017, 11:15 p.m

Review Request 61039: AMBARI-21550 PHOENIX_QUERY_SERVER not upgraded in EU

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

<    1   2   3   4   5   >