----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55840/#review162622 -----------------------------------------------------------
Ship it! Ship It! - Gaurav Nagar On Jan. 23, 2017, 11:59 a.m., Madhan Reddy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55840/ > ----------------------------------------------------------- > > (Updated Jan. 23, 2017, 11:59 a.m.) > > > Review request for Ambari, belliraj hb, Gaurav Nagar, and Rohit Choudhary. > > > Bugs: AMBARI-19674 > https://issues.apache.org/jira/browse/AMBARI-19674 > > > Repository: ambari > > > Description > ------- > > separate api for reading asset is implemented and used. Also validation for > assetName check is added > > > Diffs > ----- > > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/Constants.java > 238b002 > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/HDFSFileUtils.java > 53b43f9 > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/OozieProxyImpersonator.java > 7596de0 > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/WorkflowFilesService.java > d8bf9ff > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/assets/AssetRepo.java > df936a4 > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/assets/AssetResource.java > 0622971 > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/assets/AssetService.java > 9fe2f9c > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/workflowmanager/WorkflowManagerService.java > afdee9e > > contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/workflowmanager/WorkflowsRepo.java > 7787bda > > contrib/views/wfmanager/src/main/resources/ui/app/components/asset-config.js > dcfba34 > contrib/views/wfmanager/src/main/resources/ui/app/components/asset-list.js > 9ad0494 > > contrib/views/wfmanager/src/main/resources/ui/app/components/asset-manager.js > 4674533 > > contrib/views/wfmanager/src/main/resources/ui/app/components/designer-workspace.js > a04c66d > > contrib/views/wfmanager/src/main/resources/ui/app/components/flow-designer.js > 1822a20 > contrib/views/wfmanager/src/main/resources/ui/app/services/asset-manager.js > 3e87644 > contrib/views/wfmanager/src/main/resources/ui/app/styles/app.less d91eb3b > > contrib/views/wfmanager/src/main/resources/ui/app/templates/components/asset-config.hbs > 3ae4bff > > contrib/views/wfmanager/src/main/resources/ui/app/templates/components/asset-list.hbs > bc5201d > > contrib/views/wfmanager/src/main/resources/ui/app/templates/components/asset-manager.hbs > 7a21af1 > > contrib/views/wfmanager/src/main/resources/ui/app/templates/components/designer-workspace.hbs > 7a6005a > > contrib/views/wfmanager/src/main/resources/ui/app/templates/components/flow-designer.hbs > 38d8eaf > > Diff: https://reviews.apache.org/r/55840/diff/ > > > Testing > ------- > > manual > > > Thanks, > > Madhan Reddy > >