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

Ship it!



ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionDBAccessorImpl.java
 (line 305)
<https://reviews.apache.org/r/39418/#comment161231>

    Are we enabling assertions anywhere?  Default is false (off), so not sure 
the benefit here. (This was probably a leftover from the ported patch too)


- Nate Cole


On Oct. 20, 2015, 1:25 a.m., Robert Levas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39418/
> -----------------------------------------------------------
> 
> (Updated Oct. 20, 2015, 1:25 a.m.)
> 
> 
> Review request for Ambari, Jonathan Hurley, Nate Cole, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-13441
>     https://issues.apache.org/jira/browse/AMBARI-13441
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> This is a back-port of AMBARI-12506 to Ambari 2.1.3
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionDBAccessorImpl.java
>  30da833 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionScheduler.java
>  1e019b7 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java
>  fec76ab 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/Stage.java 
> 22b2ca4 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java
>  20b4658 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
>  27e3377 
>   
> ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostRoleCommandDAO.java
>  f04c868 
>   
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java
>  c005efc 
>   
> ambari-server/src/main/java/org/apache/ambari/server/serveraction/ServerActionExecutor.java
>  670e925 
>   
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog213.java
>  0a5a66b 
>   ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql bb0c724 
>   ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql c884d07 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 7f8b981 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql 
> 93a5dc8 
>   ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 621a524 
>   ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 72d2d25 
>   
> ambari-server/src/test/java/org/apache/ambari/server/actionmanager/StageTest.java
>  c1bd8bc 
>   
> ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionDBAccessorImpl.java
>  798261b 
>   
> ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java
>  73b1649 
>   
> ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
>  069f67d 
>   
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
>  c0c9d5e 
>   
> ambari-server/src/test/java/org/apache/ambari/server/serveraction/ServerActionExecutorTest.java
>  f9b023f 
>   
> ambari-server/src/test/java/org/apache/ambari/server/stageplanner/TestStagePlanner.java
>  c9d7856 
>   
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog213Test.java
>  20a3b1c 
> 
> Diff: https://reviews.apache.org/r/39418/diff/
> 
> 
> Testing
> -------
> 
> Manually tested:
> - 1.7.0 -> upgrade to 2.1.3 -> enable kerberos (needed to fix 
> hdfs-site/dfs.namenode.rpc-address and JCE policy on Ambari Server host)
> - 1.7.0 (ambari server host w/o agent) -> upgrade to 2.1.3 -> enable kerberos 
> (needed to fix hdfs-site/dfs.namenode.rpc-address and JCE policy on Ambari 
> Server host)
> - 2.0.2 -> upgrade to 2.1.3 -> enable kerberos
> - 2.0.2 -> enable Kerberos -> upgrade to 2.1.3 -> enable NN HA -> enable RM 
> HA -> RU to HDP 2.3.2.0
> - 2.1.2 -> upgrade to 2.1.3 -> enable kerberos
> - 2.1.2 -> enable Kerberos -> upgrade to 2.1.3
> - 2.1.3 -> enable Kerberos 
> - 2.1.3 (ambari server host w/o agent) -> enable kerberos
> 
> # Jenkins test results: PENDING
> 
> 
> Thanks,
> 
> Robert Levas
> 
>

Reply via email to