Github user mxm commented on the pull request:

    https://github.com/apache/flink/pull/858#issuecomment-138625172
  
    I've ported this pull request to the latest master. It was a lot more work 
than I anticipated because some classes had diverged significantly and merging 
them was a bit hard.
    
    Due to some refactoring, the changes have grown quite large again and I 
know that makes reviewing hard. Despite that, I wouldn't delay merging this 
pull request much further. We can disable the session management until it is 
integrated with the rest of the system (intermediate results) by throwing an 
exception on the interface methods. If we decide later, that we want to delay 
this feature, we could also remove the session code. In that case, it would 
still make sense to merge this pull request because it contains a lot of nice 
refactoring.
    
    With the session management in place, we can reuse already computed 
intermediate results with not too much effort. Actually, only some API changes 
are remaining to expose the session management to the user in production.


---
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