> On Feb. 18, 2014, 6:54 p.m., Mark Chu-Carroll wrote: > > src/main/python/apache/aurora/client/cli/options.py, line 68 > > <https://reviews.apache.org/r/18153/diff/1/?file=486455#file486455line68> > > > > This will throw a ValueError if the user didn't give a numeric value.
Good point. Fixed. - Maxim ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18153/#review34735 ----------------------------------------------------------- On Feb. 15, 2014, 12:01 a.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18153/ > ----------------------------------------------------------- > > (Updated Feb. 15, 2014, 12:01 a.m.) > > > Review request for Aurora, Mark Chu-Carroll and Brian Wickman. > > > Bugs: AURORA-207 > https://issues.apache.org/jira/browse/AURORA-207 > > > Repository: aurora > > > Description > ------- > > Added "sla get_job_uptime" command. > > Also, changed from ValueError to ArgumentTypeError in options.py as the > former is not handled properly by argparser and no custom messages were > reported. > > > Diffs > ----- > > src/main/python/apache/aurora/client/api/sla.py > b69d20818836fb6ae87b333e33008b27e013954f > src/main/python/apache/aurora/client/cli/options.py > 017f141a605eb6b522fcd7c4fe827eefdc626fca > src/main/python/apache/aurora/client/cli/sla.py > a3973df870567052083b1d5a524438191eb6d812 > src/test/python/apache/aurora/client/api/test_sla.py > 54f904c122b9e41d85dc7ad63fe3698cd0bf2923 > src/test/python/apache/aurora/client/cli/test_sla.py > 703d39e4566a30900f1e9bcca5871c4a426cc6c3 > > Diff: https://reviews.apache.org/r/18153/diff/ > > > Testing > ------- > > ./build-support/jenkins/build.sh > > > Thanks, > > Maxim Khutornenko > >
