----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46148/#review128867 -----------------------------------------------------------
Ship it! Ship It! - Oliver Szabo On April 14, 2016, 10:25 a.m., Daniel Gergely wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46148/ > ----------------------------------------------------------- > > (Updated April 14, 2016, 10:25 a.m.) > > > Review request for Ambari, Laszlo Puskas, Oliver Szabo, Robert Levas, Sandor > Magyari, and Sebastian Toader. > > > Bugs: AMBARI-15862 > https://issues.apache.org/jira/browse/AMBARI-15862 > > > Repository: ambari > > > Description > ------- > > When deleting a host, NPE may occur. > TopologyHostRequests are created with hostName=null at the first time, and > when hosts are registered, then these null values are replaced with the > correct host names. > In the case when a host is being deleted, a loop checks the host name for all > the requests, and if there is a request with hostName=null, a NPE is thrown. > Swapping the sides of **equals** method can solve the problem, since the > variable **hostname** is checked before and not null. > > Lob and Basic annotations were removed, because that column in the database > is not a CLOB. This turned out during running the test (in memory db is > created based on these annotations). > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/TopologyHostGroupEntity.java > 0a81286 > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java > c5554f7 > > ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClustersTest.java > daf8d12 > > Diff: https://reviews.apache.org/r/46148/diff/ > > > Testing > ------- > > > Thanks, > > Daniel Gergely > >