----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35078/#review86664 -----------------------------------------------------------
contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/PropertyValidator.java <https://reviews.apache.org/r/35078/#comment138721> Wouldn't a regular expression be better here? This check will allow urls like `http://ambari.server:8080/invalid/api/v1/clusers`. contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/PropertyValidator.java <https://reviews.apache.org/r/35078/#comment138723> Is this the correct error message? The check isn't validating whether the cluster nanme is in the URL. contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ViewUserLocal.java <https://reviews.apache.org/r/35078/#comment138726> This seems like a useless method. Did this method override some abstract or interface method at one point? contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ViewUserLocal.java <https://reviews.apache.org/r/35078/#comment138729> JavaDoc? contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ViewUserLocal.java <https://reviews.apache.org/r/35078/#comment138730> JavaDoc? contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ViewUserLocal.java <https://reviews.apache.org/r/35078/#comment138728> Surround with `{` and `}` contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ViewUserLocal.java <https://reviews.apache.org/r/35078/#comment138731> JavaDoc? contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ViewUserLocal.java <https://reviews.apache.org/r/35078/#comment138732> JavaDoc? contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/AmbariApi.java <https://reviews.apache.org/r/35078/#comment138733> JavaDoc? contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/ConfigurationBuilder.java <https://reviews.apache.org/r/35078/#comment138735> JavaDoc? contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/ConfigurationBuilder.java <https://reviews.apache.org/r/35078/#comment138736> JavaDoc? - Robert Levas On June 4, 2015, 1:21 p.m., Erik Bergenholtz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35078/ > ----------------------------------------------------------- > > (Updated June 4, 2015, 1:21 p.m.) > > > Review request for Ambari, Robert Levas and Tom Beerbower. > > > Bugs: AMBARI-11697 > https://issues.apache.org/jira/browse/AMBARI-11697 > > > Repository: ambari > > > Description > ------- > > - Add Automatic ResourceManager URL configuration for http/https > - Validate that Queue capacity adds up to 100% > - Allow spaces in certain configurations > - Do not allow views to be created with duplicate names > - Do not allow views to be created with empty names > > > Diffs > ----- > > contrib/views/capacity-scheduler/pom.xml 23e5aee > > contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/ConfigurationService.java > b192543 > > contrib/views/capacity-scheduler/src/main/java/org/apache/ambari/view/capacityscheduler/PropertyValidator.java > 64a8796 > > contrib/views/capacity-scheduler/src/main/resources/ui/app/components/capacityInput.js > 3f3999e > > contrib/views/capacity-scheduler/src/main/resources/ui/app/controllers/queues.js > 7b69c92 > contrib/views/capacity-scheduler/src/main/resources/ui/app/models/queue.js > 3132c10 > contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js > 4b722f9 > contrib/views/capacity-scheduler/src/main/resources/ui/app/store.js 3a61a3b > > contrib/views/capacity-scheduler/src/main/resources/ui/app/styles/application.less > 23ce2ef > > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/schedulerPanel.hbs > a5385fc > contrib/views/capacity-scheduler/src/main/resources/view.xml 564f353 > contrib/views/utils/pom.xml d56a759 > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ViewUserLocal.java > PRE-CREATION > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/AmbariApi.java > e49eea5 > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/ambari/Services.java > PRE-CREATION > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/ConfigurationBuilder.java > c3ff8bc > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/HdfsApi.java > cc329d9 > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/HdfsUtil.java > 12226ce > > contrib/views/utils/src/test/java/org/apache/ambari/view/utils/ViewUserLocalTest.java > PRE-CREATION > > contrib/views/utils/src/test/java/org/apache/ambari/view/utils/hdfs/ConfigurationBuilderTest.java > 17a9c7f > > Diff: https://reviews.apache.org/r/35078/diff/ > > > Testing > ------- > > Deploy of view and manual testing, > local unit testing > new unit tests > > > Thanks, > > Erik Bergenholtz > >
