ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Zhe (Joe) Wang" <zhewang...@gmail.com>
Subject Review Request 42376: AMBARI-14653 HBase Links Metric - more links pointing to the wrong hbase.master.info.port
Date Sat, 16 Jan 2016 00:55:41 GMT

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

Review request for Ambari, Di Li, Jaimin Jetly, Richard Zang, and Yusaku Sako.


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


Repository: ambari


Description
-------

HBase Links Metric - more links are pointing to the convetional hbase.master.info.port 60010,
while it's configured as 16010. It happens when HBase links and Yarn links metrics are both
on Dashboard.
It's due to in ambari/ambari-web/app/views/common/quick_view_link_view.js loadQuickLinksConfigSuccessCallback(),
requiredSiteNames gets overridden every time when there is a quickLinks view loaded. So when
Hbase links metric first gets loaded, it sets requiredSiteNames to ['core-site', 'hbase-site',
'hdfs-site']. But when Yarn links metric gets loaded, it overrides requiredSiteNames to ['core-site',
'hdfs-site', 'yarn-site'].
So when Yarn links get loaded, Hbase links get set to default value as 'hbase-site' is not
in requiredSiteNames.
Proposed solution is to append the new site to the list instead of override it.


Diffs
-----

  ambari-web/app/views/common/quick_view_link_view.js e58e47f 
  ambari-web/test/views/common/quick_link_view_test.js 1c985a1 

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


Testing
-------

Added unit test. Local test passed.

Manual testing done.

Attached patch through apache.


Thanks,

Zhe (Joe) Wang


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