----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57577/#review168850 -----------------------------------------------------------
Ship it! Ship It! - Yusaku Sako On March 14, 2017, 12:28 a.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57577/ > ----------------------------------------------------------- > > (Updated March 14, 2017, 12:28 a.m.) > > > Review request for Ambari, Oleg Nechiporenko, Richard Zang, Swapan Shridhar, > Xi Wang, 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 > >