Review Request 46839: SAMZA-925: support for restart control message

2016-04-29 Thread Alex Buck
I thought I would had a go at this. Very open to any feedback. Haven't written any tests as I think a significant amount of refactoring would be required to make the ConfigManager class testable. Thanks Thanks, Alex Buck

Re: Review Request 46732: SAMZA-930 fix issue with json deserialisation in YarnUtil

2016-04-29 Thread Alex Buck
------ This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46732/#review130952 --- On April 29, 2016, 11:13 a.m., Alex Buck wrote: > >

Re: Review Request 46732: SAMZA-930 fix issue with json deserialisation in YarnUtil

2016-04-29 Thread Alex Buck
it's my first review request for Samza. Thanks https://issues.apache.org/jira/browse/SAMZA-930 Thanks, Alex Buck

Re: Review Request 46732: SAMZA-930 fix issue with json deserialisation in YarnUtil

2016-04-28 Thread Alex Buck
gt; Thanks! Thanks Yi I've now added the license. - Alex --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46732/#review130871 ---

Re: Review Request 46732: SAMZA-930 fix issue with json deserialisation in YarnUtil

2016-04-28 Thread Alex Buck
e.org/jira/browse/SAMZA-930 Thanks, Alex Buck

Review Request 46732: SAMZA-930 fix issue with json deserialisation in YarnUtil

2016-04-27 Thread Alex Buck
-autoscaling/src/test/resources/exampleResourceManagerOutput.json PRE-CREATION Diff: https://reviews.apache.org/r/46732/diff/ Testing --- Very open to any feedback especially as it's my first review request for Samza. Thanks https://issues.apache.org/jira/browse/SAMZA-930 Thanks, Alex Buck