[GitHub] flink issue #5248: [FLINK-5823] [checkpoints] State backends now also handle...

[StephanEwen <gi...@git.apache.org>]

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/5248
  
    Thank you for the thorough review. Here is a summary of follow-ups from my side:
      - Investigate more if we find a better way for configuration than the `ConfigurableStateBackend`.
      - Adjust some method names.
      - Possible look at a different name for `CheckpointStorage` to avoid confusion with `CompletedCheckpointStore`. We might actually decide to rename `CompletedCheckpointStore` to `CompletedCheckpointsRegistry`, because that is really what it is. ZK registers the completed checkpoints, it does not store them (after this PR that is more true even than before).
    
    One of the bigger comments (to replace String by a stronger typed Pointer class) is something I think will not easily work, because Strings is really what we get from the command line, REST interface, ZK. Any extra wrapping would not add any more safety at this point. When we see that we actually can add more safety, we should refactor this.


---