Github user shixiaogang commented on the issue:

    https://github.com/apache/flink/pull/2768
  
    @aljoscha That way, it's very confusing that a `ReadableState` is not a 
`State`.  Hence I made `State` read-only and introduced the `UpdatableState` 
interface who extends `State` with the method `clear()`. 
    
    These changes (mainly the introduction of the `get()` method) are intended 
to remove the duplicated code. As they have little relationship with the 
implementation of map states. I think it's okay not to change these interfaces 
now.
    
    But I prefer to rethink the state hierarchy in the near future because 
there exists too much duplicated code now. 


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