Re: Review Request 62674: A couple of merge fixes for branch-3.0-perf

2017-09-29 Thread Andrew Onischuk

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

(Updated Sept. 29, 2017, 9:30 a.m.)


Review request for Ambari and Attila Doroszlai.


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


Repository: ambari


Description
---

.


Diffs (updated)
-

  ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py 
9f3e9a27a8 
  ambari-agent/src/main/python/ambari_agent/HeartbeatThread.py 2d9e5cc801 
  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapper.java
 ea208e838e 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
 5f4ba177e2 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
 d14bd33768 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
 e0df48778b 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
 5d75b9921c 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ComponentResourceProvider.java
 447d7e8fd4 
  ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RequestDAO.java 
c97f4cfc6f 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 e6da65e745 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
 8743f63aeb 
  
ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
 ecb616a6b9 
  
ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/package/templates/input.config-zeppelin.json.j2
 2b373d54e5 
  ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/config-upgrade.xml 
4a89fd4b7b 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java
 439a5542c9 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/DummyHeartbeatConstants.java
 69980dc6eb 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/HeartbeatProcessorTest.java
 f55db0831b 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
 4b4da49762 
  
ambari-server/src/test/java/org/apache/ambari/server/configuration/RecoveryConfigHelperTest.java
 1b37435a89 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java
 883e8918b6 
  
ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListenerTest.java
 ff6fab9e3b 
  
ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
 677b607834 
  
ambari-server/src/test/java/org/apache/ambari/server/state/alerts/InitialAlertEventTest.java
 3d25ef718e 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java
 95a04deb6b 
  ambari-web/app/mixins/common/persist.js e69de29bb2 


Diff: https://reviews.apache.org/r/62674/diff/3/

Changes: https://reviews.apache.org/r/62674/diff/2-3/


Testing
---

mvn clean test


Thanks,

Andrew Onischuk



Re: Review Request 62674: A couple of merge fixes for branch-3.0-perf

2017-09-28 Thread Attila Doroszlai

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


Ship it!




Ship It!

- Attila Doroszlai


On Sept. 28, 2017, 8:13 p.m., Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62674/
> ---
> 
> (Updated Sept. 28, 2017, 8:13 p.m.)
> 
> 
> Review request for Ambari and Attila Doroszlai.
> 
> 
> Bugs: AMBARI-22088
> https://issues.apache.org/jira/browse/AMBARI-22088
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> .
> 
> 
> Diffs
> -
> 
>   ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py 
> 9f3e9a27a8 
>   
> ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapper.java
>  ea208e838e 
>   
> ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
>  5f4ba177e2 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
>  d14bd33768 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
>  e0df48778b 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
>  5d75b9921c 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ComponentResourceProvider.java
>  447d7e8fd4 
>   
> ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RequestDAO.java 
> c97f4cfc6f 
>   
> ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
>  e6da65e745 
>   
> ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
>  8743f63aeb 
>   
> ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
>  ecb616a6b9 
>   
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/package/templates/input.config-zeppelin.json.j2
>  2b373d54e5 
>   ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/config-upgrade.xml 
> 4a89fd4b7b 
>   
> ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java
>  439a5542c9 
>   
> ambari-server/src/test/java/org/apache/ambari/server/agent/DummyHeartbeatConstants.java
>  69980dc6eb 
>   
> ambari-server/src/test/java/org/apache/ambari/server/agent/HeartbeatProcessorTest.java
>  f55db0831b 
>   
> ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
>  4b4da49762 
>   
> ambari-server/src/test/java/org/apache/ambari/server/configuration/RecoveryConfigHelperTest.java
>  1b37435a89 
>   
> ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java
>  883e8918b6 
>   
> ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListenerTest.java
>  ff6fab9e3b 
>   
> ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
>  677b607834 
>   
> ambari-server/src/test/java/org/apache/ambari/server/state/alerts/InitialAlertEventTest.java
>  3d25ef718e 
>   
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java
>  95a04deb6b 
> 
> 
> Diff: https://reviews.apache.org/r/62674/diff/2/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>



Re: Review Request 62674: A couple of merge fixes for branch-3.0-perf

2017-09-28 Thread Andrew Onischuk

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

(Updated Sept. 28, 2017, 6:13 p.m.)


Review request for Ambari and Attila Doroszlai.


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


Repository: ambari


Description (updated)
---

.


Diffs (updated)
-

  ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py 
9f3e9a27a8 
  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapper.java
 ea208e838e 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
 5f4ba177e2 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
 d14bd33768 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
 e0df48778b 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
 5d75b9921c 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ComponentResourceProvider.java
 447d7e8fd4 
  ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RequestDAO.java 
c97f4cfc6f 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 e6da65e745 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
 8743f63aeb 
  
ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
 ecb616a6b9 
  
ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/package/templates/input.config-zeppelin.json.j2
 2b373d54e5 
  ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/config-upgrade.xml 
4a89fd4b7b 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java
 439a5542c9 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/DummyHeartbeatConstants.java
 69980dc6eb 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/HeartbeatProcessorTest.java
 f55db0831b 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
 4b4da49762 
  
ambari-server/src/test/java/org/apache/ambari/server/configuration/RecoveryConfigHelperTest.java
 1b37435a89 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java
 883e8918b6 
  
ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListenerTest.java
 ff6fab9e3b 
  
ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
 677b607834 
  
ambari-server/src/test/java/org/apache/ambari/server/state/alerts/InitialAlertEventTest.java
 3d25ef718e 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java
 95a04deb6b 


Diff: https://reviews.apache.org/r/62674/diff/2/

Changes: https://reviews.apache.org/r/62674/diff/1-2/


Testing
---

mvn clean test


Thanks,

Andrew Onischuk



Review Request 62674: A couple of merge fixes for branch-3.0-perf

2017-09-28 Thread Andrew Onischuk

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

Review request for Ambari and Attila Doroszlai.


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


Repository: ambari


Description
---


Diffs
-

  ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py 
9f3e9a27a8 
  ambari-agent/src/main/python/ambari_agent/HeartbeatThread.py 2d9e5cc801 
  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapper.java
 ea208e838e 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
 5f4ba177e2 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
 d14bd33768 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
 e0df48778b 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
 5d75b9921c 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ComponentResourceProvider.java
 447d7e8fd4 
  ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RequestDAO.java 
c97f4cfc6f 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 e6da65e745 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
 8743f63aeb 
  
ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
 ecb616a6b9 
  
ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/package/templates/input.config-zeppelin.json.j2
 2b373d54e5 
  ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/config-upgrade.xml 
4a89fd4b7b 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java
 439a5542c9 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/DummyHeartbeatConstants.java
 69980dc6eb 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/HeartbeatProcessorTest.java
 f55db0831b 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
 4b4da49762 
  
ambari-server/src/test/java/org/apache/ambari/server/configuration/RecoveryConfigHelperTest.java
 1b37435a89 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java
 883e8918b6 
  
ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/StackVersionListenerTest.java
 ff6fab9e3b 
  
ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
 677b607834 
  
ambari-server/src/test/java/org/apache/ambari/server/state/alerts/InitialAlertEventTest.java
 3d25ef718e 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java
 95a04deb6b 


Diff: https://reviews.apache.org/r/62674/diff/1/


Testing
---

mvn clean test


Thanks,

Andrew Onischuk