----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57709/#review169577 -----------------------------------------------------------
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/capacityInput.js Line 260 (original), 260 (patched) <https://reviews.apache.org/r/57709/#comment241987> Good practice to always use braces. - Alejandro Fernandez On March 16, 2017, 10:40 p.m., Anita Jebaraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57709/ > ----------------------------------------------------------- > > (Updated March 16, 2017, 10:40 p.m.) > > > Review request for Ambari, belliraj hb, Di Li, DIPAYAN BHOWMICK, and Sangeeta > Ravindran. > > > Bugs: AMBARI-20334 > https://issues.apache.org/jira/browse/AMBARI-20334 > > > Repository: ambari > > > Description > ------- > > capacity field in yarn queue manager should allow entering decimal numbers > upto 2 places, > > currently the field allows entering values like > > 1.2.3 > 12... > 1 2.3 > > > Diffs > ----- > > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components/capacityInput.js > b6c419a > > > Diff: https://reviews.apache.org/r/57709/diff/1/ > > > Testing > ------- > > Tested manually > > > Thanks, > > Anita Jebaraj > >
