ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jaimin Jetly" <jai...@hortonworks.com>
Subject Re: Review Request 35283: With HBase master HA Ambari sometimes displays incorrect dashboard information
Date Wed, 10 Jun 2015 02:03:19 GMT

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

(Updated June 10, 2015, 2:03 a.m.)


Review request for Ambari, Srimanth Gunturi and Yusaku Sako.


Changes
-------

Removed enhanced_service_dashboard.less changes in this patch which is supposed to be part
of another patch


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


Repository: ambari


Description
-------

Behavior of service component endpoint for metric related fields inside ServiceComponent map
was undefined in HA state of the service component. It randoml picked the host component and
rolled up the metric values instead of picking up the active master host component. This mostly
affected HBase service as web ui was using those serviceComponent field.

As part of this patch, ambari-web now no longer looks at any serviceComponent fields for any
service. Instead web code figures out the active host component and gets the metric values
from the host component.


Diffs (updated)
-----

  ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/YARN_metrics.json 92c32d8

  ambari-web/app/assets/data/metrics/HBASE/definition.json 3374f2d 
  ambari-web/app/controllers/global/update_controller.js af7b69e 
  ambari-web/app/mappers/service_metrics_mapper.js 9c39fc6 
  ambari-web/app/messages.js 88c716d 
  ambari-web/app/models/service/hbase.js 662577f 
  ambari-web/app/models/service/mapreduce2.js 388b236 
  ambari-web/app/models/service/yarn.js eb33886 
  ambari-web/test/controllers/global/update_controller_test.js f55136b 
  ambari-web/test/mappers/service_mapper_test.js 1ab4cf9 

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


Testing
-------

tested the cluster dashboard and service summary pages with the patch on a live cluster with
and without HA enabled for NameNode, ResourceManager and Hbase master.

verified that all existing unit tests passess with the patch


Thanks,

Jaimin Jetly


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