[
https://issues.apache.org/jira/browse/FLINK-5778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15901273#comment-15901273
]
ASF GitHub Bot commented on FLINK-5778:
---------------------------------------
Github user uce commented on the issue:
https://github.com/apache/flink/pull/3442
I agree with the equals/hashCode question. It has been introduced for the
various involved classes by different people and not in this PR, so I think
it's best handled as part of a different issue. Would you mind opening an issue
for it?
Regarding code refactoring: I was also skeptical about the value of
refactoring this. I'm totally fine with keeping the code duplicated.
@StefanRRichter Is that OK with you as well?
> Split FileStateHandle into fileName and basePath
> ------------------------------------------------
>
> Key: FLINK-5778
> URL: https://issues.apache.org/jira/browse/FLINK-5778
> Project: Flink
> Issue Type: Sub-task
> Components: State Backends, Checkpointing
> Reporter: Ufuk Celebi
> Assignee: Ufuk Celebi
>
> Store the statePath as a basePath and a fileName and allow to overwrite the
> basePath. We cannot overwrite the base path as long as the state handle is
> still in flight and not persisted. Otherwise we risk a resource leak.
> We need this in order to be able to relocate savepoints.
> {code}
> interface RelativeBaseLocationStreamStateHandle {
> void clearBaseLocation();
> void setBaseLocation(String baseLocation);
> }
> {code}
> FileStateHandle should implement this and the SavepointSerializer should
> forward the calls when a savepoint is stored or loaded, clear before store
> and set after load.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
