Review Request 64809: AMBARI-22685 Stack OS/repos removed from cluster installation UI showed up on the Versions tab after cluster is installed

2017-12-22 Thread Di Li
, run thru cluster install. From Ambari web UI, on the Versions tab, check os/repos shown on the cluster, verify only the ones selected during the installation time remain. Thanks, Di Li

Review Request 64781: AMBARI-22680 Ambari web UI does not fail an invalid repo base URL during cluster installation

2017-12-21 Thread Di Li
during cluster install, verify the UI stops users from proceeding to the next page with an error about the URL being invalid. verify the UI moves to the next page when the urls are correct. Thanks, Di Li

Review Request 64771: AMBARI-22681 Ambari web UI failed to update stack repo directly from the Versions tab

2017-12-21 Thread Di Li
/ajax.js 3f85dfdc Diff: https://reviews.apache.org/r/64771/diff/1/ Testing --- existing JS unit tests. Patch a trunk cluster with the changes and verify the UI now can save URLs without having to go to the admin view. Thanks, Di Li

Re: Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-05 Thread Di Li
; We really shouldn't be hardcoding stack names and versions like this. > > > > what change caused this problem? there must be a better workaround > > Di Li wrote: > Hello Jonathan, > > This is the code change that caused the err >>

Re: Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-05 Thread Di Li
upgrade, verify ambari database has the repositories info in repo_version is updated from redhat7 to redhat-ppc7, verify post ambari upgrade, I could restart services without having ambari server threw exceptions. Thanks, Di Li

Re: Review Request 64344: AMBARI-22589 Ambari web UI stack version page is empty due to NPE when target stack does not contain all services from the current stack

2017-12-05 Thread Di Li
/ Testing --- patch a trunk cluster to have it be able to install a cluster with IOP 4.2.5 stack. register HDP 2.6 stack where it doesn;t have solr. verify UI, with the patch, pops up an alert dialog and also be able to show the stacks registered instead of an empty page. Thanks, Di Li

Re: Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-05 Thread Di Li
; We really shouldn't be hardcoding stack names and versions like this. > > > > what change caused this problem? there must be a better workaround > > Di Li wrote: > Hello Jonathan, > > This is the code change that caused the err >>

Re: Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-05 Thread Di Li
ps://reviews.apache.org/r/64311/diff/3/ Changes: https://reviews.apache.org/r/64311/diff/2-3/ Testing --- upgrade ambari server rpms on an IOP 4.2.5 PPC cluster, replace ambari-server jar with the one built with the fixes, run ambari-server upgrade, verify ambari database has the repositories info in repo_version is updated from redhat7 to redhat-ppc7, verify post ambari upgrade, I could restart services without having ambari server threw exceptions. Thanks, Di Li

Review Request 64344: AMBARI-22589 Ambari web UI stack version page is empty due to NPE when target stack does not contain all services from the current stack

2017-12-05 Thread Di Li
install a cluster with IOP 4.2.5 stack. register HDP 2.6 stack where it doesn;t have solr. verify UI, with the patch, pops up an alert dialog and also be able to show the stacks registered instead of an empty page. Thanks, Di Li

Re: Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-05 Thread Di Li
happen at the ambari-upgrade time and needs to deal with BigInsights 4.2.5 repositories. - Di --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64311/#review192856 --- On Dec. 4, 2017, 8:08

Re: Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-04 Thread Di Li
ps://reviews.apache.org/r/64311/diff/2/ Changes: https://reviews.apache.org/r/64311/diff/1-2/ Testing --- upgrade ambari server rpms on an IOP 4.2.5 PPC cluster, replace ambari-server jar with the one built with the fixes, run ambari-server upgrade, verify ambari database has the repositories info in repo_version is updated from redhat7 to redhat-ppc7, verify post ambari upgrade, I could restart services without having ambari server threw exceptions. Thanks, Di Li

Review Request 64311: AMBARI-22574 Failed to restart services on PPC cluster post Ambari upgrade for IOP/HDP migration

2017-12-04 Thread Di Li
Testing --- upgrade ambari server rpms on an IOP 4.2.5 PPC cluster, replace ambari-server jar with the one built with the fixes, run ambari-server upgrade, verify ambari database has the repositories info in repo_version is updated from redhat7 to redhat-ppc7, verify post ambari upgrade, I could restart services without having ambari server threw exceptions. Thanks, Di Li

Re: Review Request 64164: LZO Upgrade pre-check is missing from BigInsights Upgrade packs

2017-11-29 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64164/#review192132 --- Ship it! Ship It! - Di Li On Nov. 29, 2017, 4:35 p.m., Nate

Review Request 63877: AMBARI-22459 IOP/HDP migration NN HA restart post Ambari upgrade fails

2017-11-16 Thread Di Li
.2.5 NN HA cluster, modify the file after yum upgrade of ambari rpms to Ambari 2.6, then run ambari-server upgrade, then try to restart HDFS, verify HDFS can restart. Proceed with EU, verify EU is successful. Thanks, Di Li

Re: Review Request 63800: AMBARI-22431 Able to add config type if EU/RU of the same stack (minor version upgrade)

2017-11-14 Thread Di Li
/06efe434-a4fa-4b74-bede-7293bb6e1bf5__config_type_tests_hdp_2.6.zip Thanks, Di Li

Re: Review Request 63800: AMBARI-22431 Able to add config type if EU/RU of the same stack (minor version upgrade)

2017-11-14 Thread Di Li
-7293bb6e1bf5__config_type_tests_hdp_2.6.zip Thanks, Di Li

Review Request 63800: AMBARI-22431 Able to add config type if EU/RU of the same stack (minor version upgrade)

2017-11-14 Thread Di Li
zookeeper-fake that needs to be created and configured during EU. Thanks, Di Li

Review Request 63640: AMBARI-22378 BI 4.2.5 repoinfo misses the redhat-ppc7 section

2017-11-07 Thread Di Li
/repoinfo.xml d1b155d Diff: https://reviews.apache.org/r/63640/diff/1/ Testing --- manually tested it by patching an Ambari 2.6.0 cluster with changes Thanks, Di Li

Re: Review Request 63513: AMBARI-22337 each service should be able to implement server actions, package them add a jar to be loaded during EU

2017-11-02 Thread Di Li
-mail. To reply, visit: https://reviews.apache.org/r/63513/#review189923 --- On Nov. 2, 2017, 7:30 p.m., Di Li wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 63513: AMBARI-22337 each service should be able to implement server actions, package them add a jar to be loaded during EU

2017-11-02 Thread Di Li
s, run EU, verify the external server action classes ran and were able to provide response back to EU. Thanks, Di Li

Review Request 63513: AMBARI-22337 each service should be able to implement server actions, package them add a jar to be loaded during EU

2017-11-02 Thread Di Li
. Thanks, Di Li

Re: Review Request 62943: AMBARI-22220 - Should be able to switch the extension version to which a stack version is linked

2017-10-14 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62943/#review188067 --- Ship it! Ship It! - Di Li On Oct. 13, 2017, 6:40 p.m., Tim

Re: Review Request 62787: AMBARI-22136 Enable server_action tasks defined in EU/RU upgrade pack xml files to take parameters

2017-10-05 Thread Di Li
ositoryAction:84 - c: c 04 Oct 2017 11:32:18,590 INFO [Server Action Executor Worker 54] UpdateDesiredRepositoryAction:86 - END - Map commandParameters = getCommandParameters(); Thanks, Di Li

Re: Review Request 62787: AMBARI-22136 Enable server_action tasks defined in EU/RU upgrade pack xml files to take parameters

2017-10-05 Thread Di Li
ositoryAction:84 - c: c 04 Oct 2017 11:32:18,590 INFO [Server Action Executor Worker 54] UpdateDesiredRepositoryAction:86 - END - Map commandParameters = getCommandParameters(); Thanks, Di Li

Review Request 62787: AMBARI-22136 Enable server_action tasks defined in EU/RU upgrade pack xml files to take parameters

2017-10-05 Thread Di Li
dRepositoryAction:84 - c: c 04 Oct 2017 11:32:18,590 INFO [Server Action Executor Worker 54] UpdateDesiredRepositoryAction:86 - END - Map commandParameters = getCommandParameters(); Thanks, Di Li

Re: Review Request 62778: AMBARI-22137 - Different stack versions should be able to link to different extension versions

2017-10-05 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62778/#review187171 --- Ship it! Ship It! - Di Li On Oct. 4, 2017, 7:51 p.m., Tim

Review Request 61948: AMBARI-21830 PowerPC - IOP/HDP migration, service restart fails post Ambari upgrade

2017-08-28 Thread Di Li
g services ( generating the repo_info data in the command-[taskid].json) is the repo id "redhat-ppc7" patched a cluster with the new section in repoinfo.xml, restart Ambari ( already upgraded), veify now can restart HDFS where it can install the Hadoop client Thanks, Di Li

Re: Review Request 61875: Oozie server crashes post migration after regen kerberos keytabs and restart all services

2017-08-24 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61875/#review183734 --- Ship it! Ship It! - Di Li On Aug. 24, 2017, 8:59 a.m

Re: Review Request 61861: Post Ambari upgrade on a migration cluster Oozie failed to restart when LZO compression is enabled

2017-08-23 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61861/#review183656 --- Ship it! Ship It! - Di Li On Aug. 23, 2017, 8:25 p.m., Sid

Re: Review Request 60403: Add Kerberos HTTP SPNEGO authentication support to Ambari Metrics Monitor

2017-08-22 Thread Di Li
mp;a=commit&h=d260c89263ee0471514b4cfb8ce938134eceea12) Please close the review as needed. - Di Li On June 23, 2017, 10:49 p.m., Qin Liu wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://

Re: Review Request 61786: AMBARI-21768. Spark History Server uses wrong log dir

2017-08-21 Thread Di Li
/config-upgrade.xml Lines 379 (patched) <https://reviews.apache.org/r/61786/#comment259382> Spark2 config in IOP 4.2.5 also has these properties. They are defined under different config types though, i.e spark2-default... - Di Li On Aug. 21, 2017, 4:23 p.m., Attila Doroszlai

Re: Review Request 61548: AMBARI-21698. Error starting client components on RedHat7 and BI 4.2.5 after Ambari upgrade

2017-08-10 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61548/#review182583 --- Ship it! Ship It! - Di Li On Aug. 10, 2017, 9:43 a.m

Review Request 61415: AMBARI-21654 Post Ambari upgrade, BI value add 4.2 services should still be able to restart

2017-08-03 Thread Di Li
, their service defintions are NOT part of the stack shipped in Ambari) can restart now. Thanks, Di Li

Re: Review Request 61403: AMBARI-21554. Regenerating keytabs fails in Stack Advisor after upgrading Ambari

2017-08-03 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61403/#review182107 --- Ship it! Ship It! - Di Li On Aug. 3, 2017, 12:54 p.m

Re: Review Request 61377: Hive CLI does not work after upgrade

2017-08-02 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61377/#review181991 --- Ship it! Ship It! - Di Li On Aug. 2, 2017, 1:21 p.m

Re: Review Request 61371: AMBARI-21624. HDFS restart failed post Ambari upgrade when lzo compression was enabled

2017-08-02 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61371/#review181983 --- Ship it! Ship It! - Di Li On Aug. 2, 2017, 1:12 p.m

Re: Review Request 61371: AMBARI-21624. HDFS restart failed post Ambari upgrade when lzo compression was enabled

2017-08-02 Thread Di Li
/services/HDFS/package/scripts/balancer-emulator/balancer.log Lines 2 (patched) <https://reviews.apache.org/r/61371/#comment257808> I don't think we need the two balancer log files as part of the HDFS service definition. Please remove them if they are not necessary. - Di Li On Aug. 2,

Re: Review Request 61371: AMBARI-21624. HDFS restart failed post Ambari upgrade when lzo compression was enabled

2017-08-02 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61371/#review181973 --- Ship it! Ship It! - Di Li On Aug. 2, 2017, 9:13 a.m

Re: Review Request 61283: AMBARI-21627. Cross-stack upgrade from IOP to HDP, ranger audit properties need to be deleted

2017-08-02 Thread Di Li
/config-upgrade.xml Lines 317 (patched) <https://reviews.apache.org/r/61283/#comment257807> ranger-hive-audit exists in both BI 4.2.0 and BI 4.2.5, so the change should be in both upgrade pack xml files. - Di Li On Aug. 1, 2017, 10:34 p.m., Alejandro Fernandez

Re: Review Request 61283: AMBARI-21627. Cross-stack upgrade from IOP to HDP, ranger audit properties need to be deleted

2017-08-02 Thread Di Li
/config-upgrade.xml Lines 125 (patched) <https://reviews.apache.org/r/61283/#comment257806> ranger-kafka-audit only exists in BI 4.2.5. For BI 4.2.0, the conf change should be just the ranger-kms-audit ones. For BI 4.2.5, it should be both ranger-kafka-audit and ranger-kms-audit. -

Re: Review Request 61239: AMBARI-21611 Failed to install packages before EU on powerpc

2017-07-31 Thread Di Li
5542#file1785542line1123> > > > > Add a safeguard here in case indexOf returns -1. Only if > 0 should the > > substring method be called to get the os. > > This also assumes majorVersion is a single digit, instead look for the > > numeric characters from the en

Re: Review Request 61239: AMBARI-21611 Failed to install packages before EU on powerpc

2017-07-31 Thread Di Li
e to finish the install package step successfully Thanks, Di Li

Re: Review Request 61239: AMBARI-21611 Failed to install packages before EU on powerpc

2017-07-31 Thread Di Li
majorVersion: %s", os, majorVersion); System.out.println(result); """ OUTPUT: OS: redhat, arch: ppc, majorVersion: 100 - Di --- This is an automatically generated e-mail. To reply, visit: h

Re: Review Request 61239: AMBARI-21611 Failed to install packages before EU on powerpc

2017-07-31 Thread Di Li
eviews.apache.org/r/61239/#review181787 ------- On July 29, 2017, 2:12 a.m., Di Li wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61239/ >

Review Request 61239: AMBARI-21611 Failed to install packages before EU on powerpc

2017-07-28 Thread Di Li
one IOP x86, upgrade Ambari, patch with server jar, run install packages after registering HDP 2.6.2 stack. verify the error did not appear any more for ppc, both ppc and x86 cluster were able to finish the install package step successfully Thanks, Di Li

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-28 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61178/#review181738 --- Ship it! Ship It! - Di Li On July 28, 2017, 9:44 p.m

Re: Review Request 61214: AMBARI-21598. Spark Thrift Server stopped after upgrade due to undefined port

2017-07-28 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61214/#review181673 --- Ship it! Ship It! - Di Li On July 28, 2017, 2:50 p.m

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Di Li
> On July 27, 2017, 2:38 p.m., Di Li wrote: > > Hello Attila, > > > > The issue with the alert (2nd part of the issue), was BI stack has its own > > sparks thrift server alert that check hardcoded port 4038. HDP stack uses a > > different alert for the same

Re: Review Request 61178: AMBARI-21572. Spark configuration mismatch post express upgrade

2017-07-27 Thread Di Li
ver_status"" ), one with Critical (IOP one alert id "SPARK2_THRIFTSERVER_PROCESS", wrong, should be removed, or the port updated from 4038 to 4040) - Di Li On July 27, 2017, 2:28 p.m., Attila Doroszlai wrote: > > --

Re: Review Request 61050: AMBARI-21555 Hive restart fails to restart MySQL after Ambari upgrade against IOP 4.2.5

2017-07-24 Thread Di Li
as long as HDP yet to support MariaDB (where HDP uses mysql.d deamon instead) - Di --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61050/#review181265 ---------

Review Request 61050: AMBARI-21555 Hive restart fails to restart MySQL after Ambari upgrade against IOP 4.2.5

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

Review Request 61049: AMBARI-21553 Oozie server failed to restart post IOP 4.2 migration

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

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

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

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

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
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: > > --- &

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

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

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

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

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

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

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

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

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

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

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

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/#review179142 ------- On June 28, 2017, 6:23 p.m., Di Li wrote: > > -

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

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 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 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 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/1-2/ Testing --- to be tested later on with the integrated tests. Thanks, Di Li

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

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

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

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

  1   2   3   4   5   >