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

(Updated June 14, 2016, 7:12 p.m.)


Review request for Ambari, Ajit Kumar, Di Li, Jonathan Hurley, and Nate Cole.


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


Repository: ambari


Description
-------

Add ability to set GET request directives so that directives may be specified 
in a GET requests. For example, to limit or increase the amount of work needed 
to be performed internally to generate the data being returned.

Most data in GET requests are retrieved but filtered from the response based on 
the predicate. This means that any data the requires significant processing to 
calculate will be calculated regardless of the requested fields. By using a 
directive this can be controlled.

Note: It may be possible the use of a PropertyProvider to control this instead, 
but that may not be desired in all cases.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/api/handlers/ReadHandler.java
 95e45d6 
  ambari-server/src/main/java/org/apache/ambari/server/api/query/Query.java 
fff842a 
  ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java 
18fd94b 
  
ambari-server/src/main/java/org/apache/ambari/server/api/resources/BaseResourceDefinition.java
 dcc5217 
  
ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceDefinition.java
 8df1a02 
  
ambari-server/src/main/java/org/apache/ambari/server/api/resources/SimpleResourceDefinition.java
 394e295 
  
ambari-server/src/main/java/org/apache/ambari/server/api/resources/UpgradeResourceDefinition.java
 bf7860b 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/BaseRequest.java
 0ebeb19 
  
ambari-server/src/main/java/org/apache/ambari/server/api/services/RequestFactory.java
 19fee8e 
  
ambari-server/src/test/java/org/apache/ambari/server/api/handlers/ReadHandlerTest.java
 5cb601e 
  
ambari-server/src/test/java/org/apache/ambari/server/api/resources/BaseResourceDefinitionTest.java
 f174cb6 
  
ambari-server/src/test/java/org/apache/ambari/server/api/resources/SimpleResourceDefinitionTest.java
 6169ee7 
  
ambari-server/src/test/java/org/apache/ambari/server/api/services/RequestFactoryTest.java
 11568c9 
  
ambari-server/src/test/java/org/apache/ambari/server/audit/request/creator/AuditEventCreatorTestHelper.java
 2642418 

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


Testing
-------

# Local test results: 

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1:25:36.424s
[INFO] Finished at: Tue Jun 14 14:53:08 EDT 2016
[INFO] Final Memory: 59M/1864M
[INFO] ------------------------------------------------------------------------

# Jenkins test results: PENDING


Thanks,

Robert Levas

Reply via email to