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

Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole.


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


Repository: ambari


Description
-------

Host versions were changed for patch upgrades to allow multiple CURRENT 
repositories per host. However, it seems that a case was missed where, during a 
full upgrade, the prior repository is never set to INSTALLED. This was the 
behavior which existed previously.

STR:
- Install a cluster with ZK and HDFS on HDP 2.5
- Perform an upgrade to HDP 2.6 and Finalize

After the upgrade, the {{host_version}} table contains {{CURRENT}} entries for 
both the HDP 2.5 and 2.6 repositories. 

{code}
ambari=# select state, stack_id, version, display_name from host_version, 
repo_version WHERE host_version.repo_version_id = repo_version.repo_version_id ;
   state   | stack_id |   version    |   display_name
-----------+----------+--------------+------------------
 INSTALLED |       13 | 2.5.0.0-1237 | HDP-2.5.0.0-1237
 INSTALLED |       13 | 2.5.0.0-1237 | HDP-2.5.0.0-1237
 INSTALLED |       13 | 2.5.0.0-1237 | HDP-2.5.0.0-1237
 CURRENT   |       12 | 2.6.0.0-334  | HDP-2.6.0.0-334
 CURRENT   |       12 | 2.6.0.0-334  | HDP-2.6.0.0-334
 CURRENT   |       12 | 2.6.0.0-334  | HDP-2.6.0.0-334
(6 rows)
{code}


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostVersionDAO.java
 ad34ec7982 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostVersionEntity.java
 dbe4fe0509 
  
ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java
 475a8c9b3b 
  
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/UpgradeActionTest.java
 35fffda001 


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


Testing
-------


Thanks,

Jonathan Hurley

Reply via email to