Github user tillrohrmann commented on the issue:

    https://github.com/apache/flink/pull/2315
  
    Great work @EronWright. Code quality, testing and extent of this PR is 
really impressive :-) I think you've nicely modularized the Mesos resource 
manager with the additional actors. This makes testing much nicer :-)
    
    I only had some minor comments and questions for my own understanding. 
Tomorrow I want to try it out on a Mesos cluster to see how it works.
    
    There is only one thing I'm wondering. Since we're also currently working 
on Flip-6 where we try to put a new RPC abstraction in place in order to 
eventually remove Akka and Scala from flink-runtime, I wanted to ask whether 
you think that the FSM actors could also be replaced (sometime in the future) 
by something else? I totally agree that they are a nice abstraction for Mesos 
and allow to express the logic succinctly.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to