ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nate Cole" <nc...@hortonworks.com>
Subject Review Request 30135: Wrong stack version of Service installed
Date Wed, 21 Jan 2015 20:14:12 GMT

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

Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and John Speidel.


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


Repository: ambari


Description
-------

Issue when loading the latestUrl.  The code was checking to see if the defaultUrl was the
same as the baseUrl, and if it was, would set the baseUrl to the latest.  This caused an issue
when actually using the defaultUrl as the baseUrl.  The baseUrl was saved to the db correctly,
but since they were equal, it would always be overridden.  This meant when we sent the baseUrl
to the host to install new bits, the latest would ALWAYS be used.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java 1b7e2fe

  ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java 933f187 
  ambari-server/src/main/java/org/apache/ambari/server/state/RepositoryInfo.java 5347682 
  ambari-server/src/main/java/org/apache/ambari/server/state/stack/LatestRepoCallable.java
608637c 
  ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java
5579db3 

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


Testing
-------

Pending results.


Thanks,

Nate Cole


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message