-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50567/
-----------------------------------------------------------

Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole.


Bugs: AMBARI-17946
    https://issues.apache.org/jira/browse/AMBARI-17946


Repository: ambari


Description
-------

Intermittently, the pre-stack upgrade checks will fail with a 
{{ConcurrentModificationException}}:

{code}
 WARNING: A HTTP GET method, public javax.ws.rs.core.Response 
org.apache.ambari.server.api.services.PreUpgradeCheckService.getPreUpgradeChecks(java.lang.String,javax.ws.rs.core.HttpHeaders,javax.ws.rs.core.UriInfo),
 should not consume any entity.
27 Jul 2016 04:21:27,035 ERROR [ambari-client-thread-35] CheckHelper:120 - 
Check HOSTS_MASTER_MAINTENANCE failed
java.util.ConcurrentModificationException
        at 
java.util.LinkedHashMap$LinkedHashIterator.nextNode(LinkedHashMap.java:711)
        at 
java.util.LinkedHashMap$LinkedValueIterator.next(LinkedHashMap.java:739)
        at 
org.apache.ambari.server.checks.HostsMasterMaintenanceCheck.perform(HostsMasterMaintenanceCheck.java:83)
        at 
org.apache.ambari.server.state.CheckHelper.performChecks(CheckHelper.java:99)
        at 
org.apache.ambari.server.controller.internal.PreUpgradeCheckResourceProvider.getResources(PreUpgradeCheckResourceProvider.java:192)
        at 
org.apache.ambari.server.controller.internal.ClusterControllerImpl$ExtendedResourceProviderWrapper.queryForResources(ClusterControllerImpl.java:966)
        at 
org.apache.ambari.server.controller.internal.ClusterControllerImpl.getResources(ClusterControllerImpl.java:141)
{code}


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/api/services/PreUpgradeCheckService.java
 0525b3a 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/UpgradeService.java
 8b66491 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/UpgradePack.java
 486d432 
  
ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java
 ee0a951 

Diff: https://reviews.apache.org/r/50567/diff/


Testing
-------

mvn clean test


Thanks,

Jonathan Hurley

Reply via email to