> On March 21, 2017, 4:54 p.m., Alejandro Fernandez wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components/capacityInput.js > > Line 260 (original), 260 (patched) > > <https://reviews.apache.org/r/57709/diff/1/?file=1666450#file1666450line260> > > > > Good practice to always use braces.
I have fixed it in the updated patch - Anita ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57709/#review169577 ----------------------------------------------------------- On March 21, 2017, 5:15 p.m., Anita Jebaraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57709/ > ----------------------------------------------------------- > > (Updated March 21, 2017, 5:15 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/2/ > > > Testing > ------- > > Tested manually > > > Thanks, > > Anita Jebaraj > >
