----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51897/#review148956 -----------------------------------------------------------
Ship it! Ship It! - Zhe (Joe) Wang On Sept. 14, 2016, 7:27 p.m., Richard Zang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51897/ > ----------------------------------------------------------- > > (Updated Sept. 14, 2016, 7:27 p.m.) > > > Review request for Ambari, Andrii Tkach, Jaimin Jetly, Zhe (Joe) Wang, and > Yusaku Sako. > > > Bugs: AMBARI-18389 > https://issues.apache.org/jira/browse/AMBARI-18389 > > > Repository: ambari > > > Description > ------- > > "placeholder" only make sense when config is editable. > > > Diffs > ----- > > ambari-web/app/models/configs/objects/service_config_property.js 0c7673d > ambari-web/test/models/configs/objects/service_config_property_test.js > a6772d3 > > Diff: https://reviews.apache.org/r/51897/diff/ > > > Testing > ------- > > Manually tested on live cluster, all unit tests passed. > > > Thanks, > > Richard Zang > >