ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "DIPAYAN BHOWMICK" <dipayan.bhowm...@gmail.com>
Subject Review Request 38047: Fix Tez view to work with RM HA
Date Wed, 02 Sep 2015 08:41:20 GMT

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

Review request for Ambari, Srimanth Gunturi, Sid Wagle, and Yusaku Sako.


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


Repository: ambari


Description
-------

The code to fetch the RM Url has been refactored to use the Service Class in ambari-views-utils
module.
Services class modified to also return the ATS server Url the logic is mostly similar to fetching
the RM Url excepting connecting to the RM server(ATS server in this case)


Diffs
-----

  contrib/views/tez/pom.xml c7b9776 
  contrib/views/tez/src/main/java/org/apache/ambari/view/tez/ViewControllerImpl.java df6b468

  contrib/views/tez/src/main/resources/view.xml 91196fa 
  contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/Services.java 120e377

  contrib/views/utils/src/test/java/org/apache/ambari/view/utils/ambari/ServicesTest.java
1950df8 

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


Testing
-------

Test case added for the ATS url. Ran the unit test cases. Locally tested in VMs.


Thanks,

DIPAYAN BHOWMICK


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