[ 
https://issues.apache.org/jira/browse/FLINK-5336?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15754160#comment-15754160
 ] 

ASF GitHub Bot commented on FLINK-5336:
---------------------------------------

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/3018
  
    Unfortunately, this breaks the API's binary compatibility. That's why the 
issue is also a sub-issue of the Flink 2.0 breaking changes. I think we need to 
delay this until we start the 2.0 branch.


> Make Path immutable
> -------------------
>
>                 Key: FLINK-5336
>                 URL: https://issues.apache.org/jira/browse/FLINK-5336
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Core
>            Reporter: Stephan Ewen
>            Assignee: Anton Solovev
>             Fix For: 2.0.0
>
>
> The {{Path}} class is currently mutable to support the {{IOReadableWritable}} 
> serialization. Since that serialization is not used any more, I suggest to 
> drop that interface from Path and make the Path's URI final.
> Being immutable, we can store configures paths properly without the chance of 
> them being mutated as side effects.
> Many parts of the code make the assumption that the Path is immutable, being 
> susceptible to subtle errors.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to