> On Nov. 28, 2014, 2:10 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/HostStackVersionService.java, > > line 117 > > <https://reviews.apache.org/r/28405/diff/3/?file=777883#file777883line117> > > > > What about if just this.clusterName == null?
I've used another class as an example. Your suggestion is good, will fix both classes. - Dmitro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28405/#review63253 ----------------------------------------------------------- On Nov. 27, 2014, 8:26 p.m., Dmitro Lisnichenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28405/ > ----------------------------------------------------------- > > (Updated Nov. 27, 2014, 8:26 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Nate Cole, and Yurii Shylov. > > > Bugs: AMBARI-8434 > https://issues.apache.org/jira/browse/AMBARI-8434 > > > Repository: ambari > > > Description > ------- > > Preview version of patch. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/services/ClusterStackVersionService.java > ed94877 > > ambari-server/src/main/java/org/apache/ambari/server/api/services/HostService.java > c4ad169 > > ambari-server/src/main/java/org/apache/ambari/server/api/services/HostStackVersionService.java > 86a00a8 > > ambari-server/src/main/java/org/apache/ambari/server/api/services/RequestService.java > 95d8963 > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementController.java > a1ece2c > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java > 0e65a1d > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java > 0fc590e > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/DefaultProviderModule.java > 83f0e0f > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostStackVersionResourceProvider.java > 67e62d7 > > ambari-server/src/main/java/org/apache/ambari/server/controller/spi/NoSuchParentResourceException.java > 308841f > > ambari-server/src/main/java/org/apache/ambari/server/controller/spi/ResourceProvider.java > de1c9dd > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > 850f079 > > ambari-server/src/test/java/org/apache/ambari/server/api/services/HostStackVersionServiceTest.java > 960a485 > ambari-server/src/test/python/custom_actions/TestInstallPackages.py 51fc9e1 > > ambari-server/src/test/python/custom_actions/configs/install_packages_config.json > 1055a40 > > Diff: https://reviews.apache.org/r/28405/diff/ > > > Testing > ------- > > manual verification on 2-node cluster > > proper unit test coverage is pending > > > File Attachments > ---------------- > > patch_27.11 > > https://reviews.apache.org/media/uploaded/files/2014/11/27/3565983c-b9ec-4dc4-94de-cd0b875ccdc4__a2.patch > > > Thanks, > > Dmitro Lisnichenko > >
