GitHub PushEvent at 2014-10-23 17:25:09
Repository: incubator-twill
Actor: chtyim
Payload: {"shas":[["d808ffdbe54b19b6433514271d6dc7b1a514f8bb","124a74c257b731c60c274382144d5c07e3dcf301@apache.org","(TWILL-105) Remove ZKDecoratorService and some pom file fixes\n\n- The ZKDecoratorSerivce is complicates and is not necessarily and have complicated race condition inside\n- Most of the functionalities in ZKDecoratorService are moved into AbstractTwillService\n  - It uses AbstractExecutionThreadService from guava that simplifies a lot of service+zk related interactions\n  - Remove use of StateNode and updating of Service states in ZK as no one is actually using it\n- The Hadoop secure store related code in AbstractTwillService are moved into AbstractYarnTwillService\n- Fixes in the pom.xml\n  - There should be one Hadoop version profile that is activeByDefault\n  - Increase memory size for unit-test\n- Fix YARN test cases\n  - Increase test timeout for PolicyPlacementTest\n  - Correct assertion in ResourceReportTestRun.testResourceReportWithFailingContainers","Terence Yim",true]],"size":1,"ref":"refs/heads/feature/fix-zk-decorator-race","head":"d808ffdbe54b19b6433514271d6dc7b1a514f8bb"}