You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Di Li <di...@ca.ibm.com> on 2015/10/28 16:38:56 UTC

Review Request 39721: AMBARI-13587: RU: Hive metastore backup manual step, a bit clear msg

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39721/
-----------------------------------------------------------

Review request for Ambari and Alejandro Fernandez.


Bugs: AMBARI-13587
    https://issues.apache.org/jira/browse/AMBARI-13587


Repository: ambari


Description
-------

The message currently reads

"Before continuing, please backup the Hive Metastore database located on the following host(s): hosts.all."

The hive metastore may be configured against external db that is not on a hive node. Proposing to change the message to "Before continuing, please backup the Hive Metastore database referenced by Hive Metastore service(s) on the following host(s): {{hosts.all}}."


Diffs
-----

  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml c283391 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml c091bb7 
  ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml ea2c63f 

Diff: https://reviews.apache.org/r/39721/diff/


Testing
-------

Manually patch a cluster and run rolling upgrade.


Thanks,

Di Li


Re: Review Request 39721: AMBARI-13587: RU: Hive metastore backup manual step, a bit clear msg

Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39721/#review104576
-----------------------------------------------------------

Ship it!


Ship It!

- Alejandro Fernandez


On Oct. 28, 2015, 3:38 p.m., Di Li wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39721/
> -----------------------------------------------------------
> 
> (Updated Oct. 28, 2015, 3:38 p.m.)
> 
> 
> Review request for Ambari and Alejandro Fernandez.
> 
> 
> Bugs: AMBARI-13587
>     https://issues.apache.org/jira/browse/AMBARI-13587
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> The message currently reads
> 
> "Before continuing, please backup the Hive Metastore database located on the following host(s): hosts.all."
> 
> The hive metastore may be configured against external db that is not on a hive node. Proposing to change the message to "Before continuing, please backup the Hive Metastore database referenced by Hive Metastore service(s) on the following host(s): {{hosts.all}}."
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml c283391 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml c091bb7 
>   ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml ea2c63f 
> 
> Diff: https://reviews.apache.org/r/39721/diff/
> 
> 
> Testing
> -------
> 
> Manually patch a cluster and run rolling upgrade.
> 
> 
> Thanks,
> 
> Di Li
> 
>


Re: Review Request 39721: AMBARI-13587: RU: Hive metastore backup manual step, a bit clear msg

Posted by Di Li <di...@ca.ibm.com>.

> On Oct. 30, 2015, 6:58 p.m., Alejandro Fernandez wrote:
> > FYI, the non-rolling upgrade packs contain that string as well.
> > Can you create another patch?

Hello Alejandro,

I just uploaded a new patch with the 2.1/2.2/2.3 nonrolling_upgrade.xml files changes.

Please review.

Thanks.

Di


- Di


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39721/#review104603
-----------------------------------------------------------


On Oct. 28, 2015, 3:38 p.m., Di Li wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39721/
> -----------------------------------------------------------
> 
> (Updated Oct. 28, 2015, 3:38 p.m.)
> 
> 
> Review request for Ambari and Alejandro Fernandez.
> 
> 
> Bugs: AMBARI-13587
>     https://issues.apache.org/jira/browse/AMBARI-13587
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> The message currently reads
> 
> "Before continuing, please backup the Hive Metastore database located on the following host(s): hosts.all."
> 
> The hive metastore may be configured against external db that is not on a hive node. Proposing to change the message to "Before continuing, please backup the Hive Metastore database referenced by Hive Metastore service(s) on the following host(s): {{hosts.all}}."
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml c283391 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml c091bb7 
>   ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml ea2c63f 
> 
> Diff: https://reviews.apache.org/r/39721/diff/
> 
> 
> Testing
> -------
> 
> Manually patch a cluster and run rolling upgrade.
> 
> 
> Thanks,
> 
> Di Li
> 
>


Re: Review Request 39721: AMBARI-13587: RU: Hive metastore backup manual step, a bit clear msg

Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39721/#review104603
-----------------------------------------------------------


FYI, the non-rolling upgrade packs contain that string as well.
Can you create another patch?

- Alejandro Fernandez


On Oct. 28, 2015, 3:38 p.m., Di Li wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39721/
> -----------------------------------------------------------
> 
> (Updated Oct. 28, 2015, 3:38 p.m.)
> 
> 
> Review request for Ambari and Alejandro Fernandez.
> 
> 
> Bugs: AMBARI-13587
>     https://issues.apache.org/jira/browse/AMBARI-13587
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> The message currently reads
> 
> "Before continuing, please backup the Hive Metastore database located on the following host(s): hosts.all."
> 
> The hive metastore may be configured against external db that is not on a hive node. Proposing to change the message to "Before continuing, please backup the Hive Metastore database referenced by Hive Metastore service(s) on the following host(s): {{hosts.all}}."
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml c283391 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml c091bb7 
>   ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml ea2c63f 
> 
> Diff: https://reviews.apache.org/r/39721/diff/
> 
> 
> Testing
> -------
> 
> Manually patch a cluster and run rolling upgrade.
> 
> 
> Thanks,
> 
> Di Li
> 
>


Re: Review Request 39721: AMBARI-13587: RU: Hive metastore backup manual step, a bit clear msg

Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39721/#review104609
-----------------------------------------------------------


Thanks, I'm also going to submit this to branch 2.1 so it makes 2.1.3 release

- Alejandro Fernandez


On Oct. 30, 2015, 7:32 p.m., Di Li wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39721/
> -----------------------------------------------------------
> 
> (Updated Oct. 30, 2015, 7:32 p.m.)
> 
> 
> Review request for Ambari and Alejandro Fernandez.
> 
> 
> Bugs: AMBARI-13587
>     https://issues.apache.org/jira/browse/AMBARI-13587
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> The message currently reads
> 
> "Before continuing, please backup the Hive Metastore database located on the following host(s): hosts.all."
> 
> The hive metastore may be configured against external db that is not on a hive node. Proposing to change the message to "Before continuing, please backup the Hive Metastore database referenced by Hive Metastore service(s) on the following host(s): {{hosts.all}}."
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/stacks/HDP/2.1/upgrades/nonrolling-upgrade-2.3.xml a60c891 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.2.xml 6c91901 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.3.xml cae5912 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml 2c413a0 
>   ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml d75449a 
>   ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.3.xml a579e54 
>   ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml 976742f 
> 
> Diff: https://reviews.apache.org/r/39721/diff/
> 
> 
> Testing
> -------
> 
> Manually patch a cluster and run rolling upgrade.
> 
> 
> Thanks,
> 
> Di Li
> 
>


Re: Review Request 39721: AMBARI-13587: RU: Hive metastore backup manual step, a bit clear msg

Posted by Di Li <di...@ca.ibm.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39721/
-----------------------------------------------------------

(Updated Oct. 30, 2015, 7:32 p.m.)


Review request for Ambari and Alejandro Fernandez.


Changes
-------

new patch file to include the message changes for the nonrolling_upgrade.xml files.


Bugs: AMBARI-13587
    https://issues.apache.org/jira/browse/AMBARI-13587


Repository: ambari


Description
-------

The message currently reads

"Before continuing, please backup the Hive Metastore database located on the following host(s): hosts.all."

The hive metastore may be configured against external db that is not on a hive node. Proposing to change the message to "Before continuing, please backup the Hive Metastore database referenced by Hive Metastore service(s) on the following host(s): {{hosts.all}}."


Diffs (updated)
-----

  ambari-server/src/main/resources/stacks/HDP/2.1/upgrades/nonrolling-upgrade-2.3.xml a60c891 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.2.xml 6c91901 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.3.xml cae5912 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml 2c413a0 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml d75449a 
  ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.3.xml a579e54 
  ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml 976742f 

Diff: https://reviews.apache.org/r/39721/diff/


Testing
-------

Manually patch a cluster and run rolling upgrade.


Thanks,

Di Li