----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28620/#review63625 -----------------------------------------------------------
Overall I think we need to rewrite the admin layer, its too much work to add any new command. And most of the code is copy paste, we should be able to solve it with a simple config. - Kishore Gopalakrishna On Dec. 3, 2014, 1:12 a.m., Zhen Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28620/ > ----------------------------------------------------------- > > (Updated Dec. 3, 2014, 1:12 a.m.) > > > Review request for helix and Shi Lu. > > > Bugs: HELIX-546 > > > Repository: helix-git > > > Description > ------- > > Add javadoc to job queue management REST API > > > Diffs > ----- > > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/ClusterResource.java > b22d801 > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/ClustersResource.java > 4b12054 > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/ConfigResource.java > 3c384d4 > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/JobQueueResource.java > 3ff9a37 > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/JobQueuesResource.java > 24a4387 > > helix-admin-webapp/src/main/java/org/apache/helix/webapp/resources/JobResource.java > a58e223 > > Diff: https://reviews.apache.org/r/28620/diff/ > > > Testing > ------- > > mvn test > > > Thanks, > > Zhen Zhang > >
