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

(Updated Feb. 19, 2014, 3:13 p.m.)


Review request for Twill and Terence Yim.


Changes
-------

Review issues resolved


Summary (updated)
-----------------

made changes as per review #18209 for #TWILL-40


Bugs: TWILL-40
    https://issues.apache.org/jira/browse/TWILL-40


Repository: twill


Description (updated)
-------

made changes as per review #18209 for #TWILL-40


Merge branch 'master' of 
https://git-wip-us.apache.org/repos/asf/incubator-twill into figureout


fixing checkstyle violations #TWILL-40


Merge branch 'master' of 
https://git-wip-us.apache.org/repos/asf/incubator-twill into figureout


exposes the YARN client ContainerRequest hosts and racks options into 
ResourceSpecification #TWILL-40


Diffs (updated)
-----

  pom.xml 214100077e36f40736216433aa0c3791b706afdb 
  twill-api/src/main/java/org/apache/twill/api/ResourceSpecification.java 
0bab8110c3662e5bf87f3e0ac6cee60e77cccf60 
  
twill-api/src/main/java/org/apache/twill/internal/DefaultResourceSpecification.java
 1327ce56aff5c9ca481afc65086950648974075f 
  twill-core/pom.xml 859c65335afe3adfebcaace249174ecf1d197df3 
  
twill-core/src/main/java/org/apache/twill/internal/json/ResourceSpecificationCodec.java
 d3b9707ca9afa058ea0c3b135886cfd22d7ed9db 
  
twill-core/src/test/java/org/apache/twill/internal/json/ResourceSpecificationCodecTest.java
 PRE-CREATION 
  
twill-yarn/src/main/java/org/apache/twill/internal/appmaster/ApplicationMasterService.java
 7caedad42fd6b4c2c32153826e6e48db6613df80 
  twill-yarn/src/main/java/org/apache/twill/internal/yarn/YarnAMClient.java 
b0dbce0ebfb990950fc9cf2d5d63914bea7f14e4 

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


Testing
-------


Thanks,

Fabian Murariu

Reply via email to