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



Saw a handful of test errors:


Tests in error:
  ExecutionCommandWrapperTest.testGetExecutionCommand:213 » NullPointer
  AgentResourceTest.setUp:124 » Creation Guice creation errors:

1) No implement...
  AgentResourceTest.setUp:124 » Creation Guice creation errors:

1) No implement...
  AgentResourceTest.setUp:124 » Creation Guice creation errors:

1) No implement...
  AgentResourceTest.setUp:124 » Creation Guice creation errors:

1) No implement...
  AgentResourceTest.setUp:124 » Creation Guice creation errors:

1) No implement...
  UpgradeResourceProviderHDP22Test.testCreateIntraStackUpgrade:371 » NullPointer
  StageUtilsTest.testGetATestStage:156 » Configuration Guice configuration 
error...
  StageUtilsTest.testJasonToExecutionCommand:183 » Configuration Guice 
configura...
  StageUtilsTest.testJaxbToString:169 » Configuration Guice configuration 
errors...

- Nate Cole


On April 28, 2016, 4:46 p.m., Jonathan Hurley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46797/
> -----------------------------------------------------------
> 
> (Updated April 28, 2016, 4:46 p.m.)
> 
> 
> Review request for Ambari, Nate Cole and Sid Wagle.
> 
> 
> Bugs: AMBARI-16082
>     https://issues.apache.org/jira/browse/AMBARI-16082
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Recent changes to the AmbariManagementControllerTest began reusing the same 
> database for all test cases. This significantly dropped the test time from 5+ 
> minutes to about 40 seconds. 
> 
> However, there is a very bad anti-pattern in our code which caused a 
> different, static Injector to live inside of the ExecutionCommandWrapper. We 
> should be utilizing proper Guice factory patterns here to create instances of 
> injected objects.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapper.java
>  52febc4 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapperFactory.java
>  PRE-CREATION 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java
>  83fa6b9 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommandFactoryImpl.java
>  cf1e989 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/Stage.java 
> 5a313d8 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/StageFactoryImpl.java
>  9ee7c16 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/ControllerModule.java
>  91d6b4d 
>   
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/CalculatedStatusTest.java
>  46c0b03 
>   
> ambari-server/src/test/java/org/apache/ambari/server/controller/internal/TaskResourceProviderTest.java
>  91cfef8 
>   
> ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/AutoSkipFailedSummaryActionTest.java
>  5e35b301 
> 
> Diff: https://reviews.apache.org/r/46797/diff/
> 
> 
> Testing
> -------
> 
> Tested AMCTest as well as the others changed.
> 
> 
> Thanks,
> 
> Jonathan Hurley
> 
>

Reply via email to