ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitro Lisnichenko" <dlysniche...@hortonworks.com>
Subject Re: Review Request 20730: Introduce operation_level request parameter
Date Fri, 25 Apr 2014 20:23:03 GMT

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

(Updated April 25, 2014, 8:22 p.m.)


Review request for Ambari, Sumit Mohanty and Sid Wagle.


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


Repository: ambari


Description
-------

Add Requests/operation_level property support and support of subproperties. Possible "level"
values are CLUSTER|SERVICE|HOST|HOST_COMPONENT. I've removed COMPONENT level (specified at
https://docs.google.com/a/hortonworks.com/document/d/1IB-2NGpLC3pb25S0FBI6L8PKaE24SMFqiJJKQHw9h8Q/edit#)
from the list of available values, because it seems to be identical to SERVICE level.

This is a preview version of patch.

Patch seems to work, but unit test fixing/coverage is not finished yet


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/actionmanager/Request.java e209076

  ambari-server/src/main/java/org/apache/ambari/server/agent/ExecutionCommand.java 40e9abb

  ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatMonitor.java 7452a7b

  ambari-server/src/main/java/org/apache/ambari/server/api/services/parsers/JsonRequestBodyParser.java
3443b72 
  ambari-server/src/main/java/org/apache/ambari/server/api/services/parsers/RequestBodyParser.java
aefcb52 
  ambari-server/src/main/java/org/apache/ambari/server/controller/ActionExecutionContext.java
37a404f 
  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java
89eaa40 
  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
de3db69 
  ambari-server/src/main/java/org/apache/ambari/server/controller/ExecuteActionRequest.java
3b5afcd 
  ambari-server/src/main/java/org/apache/ambari/server/controller/MaintenanceStateHelper.java
23583b5 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestOperationLevel.java
PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java
36def40 
  ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ClusterStateEntity.java
14f0d55 
  ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RequestEntity.java e7098d2

  ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RequestOperationLevelEntity.java
PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/state/State.java 835b751 
  ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog160.java fb4a87a

  ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql bac1615 
  ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 2b7f4d5 
  ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 0a7a432 
  ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql cbf8579 
  ambari-server/src/main/resources/META-INF/persistence.xml 36cafe2 
  ambari-server/src/main/resources/properties.json f92d238 
  ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionDBAccessorImpl.java
5ad1e77 
  ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
ff3f08a 
  ambari-server/src/test/java/org/apache/ambari/server/controller/MaintenanceStateHelperTest.java
5fc66a6 
  ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog160Test.java
062338b 

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


Testing
-------

Patch seems to work, but unit test fixing/coverage is not finished yet


Thanks,

Dmitro Lisnichenko


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message