----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31624/#review74746 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostComponentStateDAO.java <https://reviews.apache.org/r/31624/#comment121439> I think it would be nicer to have the remove method as part of the entity so that we don't need to worry about concurrent modification exceptions. What if something is iterating while this remove happens? Something we could track in a different Jira if you didn't want to fix it now. - Jonathan Hurley On March 2, 2015, 10:38 a.m., Tom Beerbower wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31624/ > ----------------------------------------------------------- > > (Updated March 2, 2015, 10:38 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Nate > Cole. > > > Bugs: AMBARI-9869 > https://issues.apache.org/jira/browse/AMBARI-9869 > > > Repository: ambari > > > Description > ------- > > Finalize fails because one of the hosts (always the one that had the > secondary namenode before) still has a host_version with a state of UPGRADING > and doesn't transition it to UPGRADED. > > When the SECONDARY_NAMENODE is deleted via the Namenode HA wizard, the > HostEntity may still retain a reference to it, thereby causing the object to > remain in memory. > > For some reason the bi-directional relationships between the host entity and > the state entities are not cleaned up on the host side by JPA. > > The patch uses a brute force approach to remove the state entites from the > collections held by the host entity. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostComponentDesiredStateDAO.java > 9635f30 > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostComponentStateDAO.java > 66e91d3 > > ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java > fe5397b > > Diff: https://reviews.apache.org/r/31624/diff/ > > > Testing > ------- > > Manual testing to verify that the HostEntity references to the state entities > are cleaned up. > > All existing tests pass ... > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 33:16 min > [INFO] Finished at: 2015-03-02T10:37:09-05:00 > [INFO] Final Memory: 42M/568M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Tom Beerbower > >
