Re: Review Request 59941: Upgrade PreUpgradeCheck NullPointerException

2017-06-09 Thread Jonathan Hurley

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


Ship it!




Interesting - how did you come across this NPE? Every upgrade pack I've seen 
has some built-in pre-checks which are required.

- Jonathan Hurley


On June 9, 2017, 1:59 a.m., wang yaoxin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59941/
> ---
> 
> (Updated June 9, 2017, 1:59 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Jonathan Hurley.
> 
> 
> Bugs: AMBARI-21208
> https://issues.apache.org/jira/browse/AMBARI-21208
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> java.lang.NullPointerException
> at 
> org.apache.ambari.server.state.stack.UpgradePack.getPrerequisiteChecks(UpgradePack.java:151)
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/state/stack/UpgradePack.java
>  b8e57c7 
> 
> 
> Diff: https://reviews.apache.org/r/59941/diff/1/
> 
> 
> Testing
> ---
> 
> Done!
> 
> 
> Thanks,
> 
> wang yaoxin
> 
>



Review Request 59941: Upgrade PreUpgradeCheck NullPointerException

2017-06-08 Thread wang yaoxin

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

Review request for Ambari, Alejandro Fernandez and Jonathan Hurley.


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


Repository: ambari


Description
---

java.lang.NullPointerException
at 
org.apache.ambari.server.state.stack.UpgradePack.getPrerequisiteChecks(UpgradePack.java:151)


Diffs
-

  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/UpgradePack.java
 b8e57c7 


Diff: https://reviews.apache.org/r/59941/diff/1/


Testing
---

Done!


Thanks,

wang yaoxin