You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@ambari.apache.org by Jaimin Jetly <ja...@hortonworks.com> on 2017/03/14 00:26:26 UTC

Review Request 57577: ambari-web shows redundant configuration for "Number of nodes used by Hive's LLAP"

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

Review request for Ambari, Oleg Nechiporenko, Swapan Shridhar, and Yusaku Sako.


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


Repository: ambari


Description
-------

*STR:*
When the queue selected for LLAP not being managed by SA is saved, on navigating to Hive smart config page, "Number of nodes used by Hive's LLAP" shows both textbox and slider. This is because the observer for LLAP widget is not fired for the first time when readonly is true to begin with.


Diffs
-----

  ambari-web/app/mixins/common/configs/widgets/unique/num_llap_nodes.js 781a40a 


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


Testing
-------

Tested the patch on a repro cluster.
veified that all ambari-web unit tests passes:

  30367 passing (21s)
  157 pending


Thanks,

Jaimin Jetly