[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-06-26 Thread StefanRRichter
Github user StefanRRichter commented on the issue:

https://github.com/apache/flink/pull/5777
  
LGTM 👍 Will merge.


---


[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-06-19 Thread yanghua
Github user yanghua commented on the issue:

https://github.com/apache/flink/pull/5777
  
@StefanRRichter , does this PR look good to you?


---


[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-06-13 Thread yanghua
Github user yanghua commented on the issue:

https://github.com/apache/flink/pull/5777
  
cc @zentol this PR takes a long time, can you review this?


---


[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-05-07 Thread yanghua
Github user yanghua commented on the issue:

https://github.com/apache/flink/pull/5777
  
cc @zentol @GJL @tzulitai this PR takes a long time, please review it 
thanks.


---


[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-04-25 Thread yanghua
Github user yanghua commented on the issue:

https://github.com/apache/flink/pull/5777
  
cc @zentol 


---


[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-04-19 Thread yanghua
Github user yanghua commented on the issue:

https://github.com/apache/flink/pull/5777
  
cc @zentol 


---


[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-04-09 Thread yuqi1129
Github user yuqi1129 commented on the issue:

https://github.com/apache/flink/pull/5777
  
@tedyu , You are right, I ignore the code below which do not show in the 
change. 
+1


---


[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-04-09 Thread tedyu
Github user tedyu commented on the issue:

https://github.com/apache/flink/pull/5777
  
@yuqi1129 
What do you think ?


---


[GitHub] flink issue #5777: [FLINK-7897] Consider using nio.Files for file deletion i...

2018-03-27 Thread yanghua
Github user yanghua commented on the issue:

https://github.com/apache/flink/pull/5777
  
cc @tillrohrmann 


---