> On April 21, 2017, 5:45 a.m., Attila Doroszlai wrote: > > ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java > > Lines 24-25 (patched) > > <https://reviews.apache.org/r/58590/diff/1/?file=1695882#file1695882line24> > > > > Unused imports
Thanks. I've removed the unused imports. - Sangeeta ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58590/#review172586 ----------------------------------------------------------- On April 24, 2017, 11:33 p.m., Sangeeta Ravindran wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58590/ > ----------------------------------------------------------- > > (Updated April 24, 2017, 11:33 p.m.) > > > Review request for Ambari, Di Li and Tim Thorpe. > > > Bugs: AMBARI-20616 > https://issues.apache.org/jira/browse/AMBARI-20616 > > > Repository: ambari > > > Description > ------- > > Some services in our stack have custom commands where the name is a > combination of an operation and a component name. > > For e.g. <name>Remove_Logical_myservice_mycomponent</name> > > When the custom command is being executed, the operations window title is > displayed as "Remove Logical myservice mycomponent mycomponent" > > It would be useful to provide an optional attribute e.g. opsDisplayName for > custom commands to allow specifying the string for the title displayed in the > operations window. > > <customCommand> > <name>Remove_Logical_myservice_mycomponent</name> > <opsDisplayName>true</opsDisplayName> > <commandScript> > <script>scripts/mycomponent.py</script> > <scriptType>PYTHON</scriptType> > <timeout>10800</timeout> > </commandScript> > </customCommand> > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java > 87a6edf > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java > 17bc718 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java > 20b0417 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java > 86feceb > > ambari-server/src/main/java/org/apache/ambari/server/state/CustomCommandDefinition.java > 280a59b > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java > 633d837 > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql fd49b94 > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9fc3209 > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 310208d > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql c052104 > ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 5a58ef8 > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8c7a31d > ambari-server/src/main/resources/properties.json c2545fe > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java > 8a52b8a > > ambari-server/src/test/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntityTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java > fec041c > ambari-web/app/controllers/wizard/step9_controller.js 4697dad > ambari-web/app/utils/ajax/ajax.js 2b5e52a > ambari-web/app/utils/helper.js 03a2e82 > ambari-web/app/utils/host_progress_popup.js c615cae > ambari-web/app/views/wizard/step9/hostLogPopupBody_view.js d983252 > ambari-web/test/utils/helper_test.js 4b9ec36 > > > Diff: https://reviews.apache.org/r/58590/diff/2/ > > > Testing > ------- > > Manual Testing. > Added new test cases. > > > Thanks, > > Sangeeta Ravindran > >