[jira] [Commented] (FLINK-2377) AbstractTestBase.deleteAllTempFiles sometimes fails on Windows

2015-07-21 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/FLINK-2377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14635631#comment-14635631
 ] 

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

Github user asfgit closed the pull request at:

https://github.com/apache/flink/pull/924


 AbstractTestBase.deleteAllTempFiles sometimes fails on Windows
 --

 Key: FLINK-2377
 URL: https://issues.apache.org/jira/browse/FLINK-2377
 Project: Flink
  Issue Type: Bug
  Components: Tests
 Environment: Windows
Reporter: Gabor Gevay
Priority: Minor

 This is probably another file closing issue. (that is, Windows won't delete 
 open files, as opposed to Linux)
 I have encountered two concrete tests so far where this actually appears: 
 CsvOutputFormatITCase and CollectionSourceTest.



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


[jira] [Commented] (FLINK-2377) AbstractTestBase.deleteAllTempFiles sometimes fails on Windows

2015-07-21 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/FLINK-2377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14635169#comment-14635169
 ] 

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

Github user StephanEwen commented on the pull request:

https://github.com/apache/flink/pull/924#issuecomment-123326390
  
+1, will merge this...


 AbstractTestBase.deleteAllTempFiles sometimes fails on Windows
 --

 Key: FLINK-2377
 URL: https://issues.apache.org/jira/browse/FLINK-2377
 Project: Flink
  Issue Type: Bug
  Components: Tests
 Environment: Windows
Reporter: Gabor Gevay
Priority: Minor

 This is probably another file closing issue. (that is, Windows won't delete 
 open files, as opposed to Linux)
 I have encountered two concrete tests so far where this actually appears: 
 CsvOutputFormatITCase and CollectionSourceTest.



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


[jira] [Commented] (FLINK-2377) AbstractTestBase.deleteAllTempFiles sometimes fails on Windows

2015-07-20 Thread Stephan Ewen (JIRA)

[ 
https://issues.apache.org/jira/browse/FLINK-2377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14633162#comment-14633162
 ] 

Stephan Ewen commented on FLINK-2377:
-

Can you verify that this is a problem of the test, i.e., the test forgets to 
close the files?

 AbstractTestBase.deleteAllTempFiles sometimes fails on Windows
 --

 Key: FLINK-2377
 URL: https://issues.apache.org/jira/browse/FLINK-2377
 Project: Flink
  Issue Type: Bug
  Components: Tests
 Environment: Windows
Reporter: Gabor Gevay
Priority: Minor

 This is probably another file closing issue. (that is, Windows won't delete 
 open files, as opposed to Linux)
 I have encountered two concrete tests so far where this actually appears: 
 CsvOutputFormatITCase and CollectionSourceTest.



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


[jira] [Commented] (FLINK-2377) AbstractTestBase.deleteAllTempFiles sometimes fails on Windows

2015-07-20 Thread Gabor Gevay (JIRA)

[ 
https://issues.apache.org/jira/browse/FLINK-2377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14633525#comment-14633525
 ] 

Gabor Gevay commented on FLINK-2377:


Yes, I have found the problem: TestBaseUtils.readAllResultLines is not closing 
the readers. I will open a PR that fixes this.

 AbstractTestBase.deleteAllTempFiles sometimes fails on Windows
 --

 Key: FLINK-2377
 URL: https://issues.apache.org/jira/browse/FLINK-2377
 Project: Flink
  Issue Type: Bug
  Components: Tests
 Environment: Windows
Reporter: Gabor Gevay
Priority: Minor

 This is probably another file closing issue. (that is, Windows won't delete 
 open files, as opposed to Linux)
 I have encountered two concrete tests so far where this actually appears: 
 CsvOutputFormatITCase and CollectionSourceTest.



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


[jira] [Commented] (FLINK-2377) AbstractTestBase.deleteAllTempFiles sometimes fails on Windows

2015-07-20 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/FLINK-2377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14633628#comment-14633628
 ] 

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

Github user chiwanpark commented on the pull request:

https://github.com/apache/flink/pull/924#issuecomment-122893574
  
Great catch! Looks good to merge.


 AbstractTestBase.deleteAllTempFiles sometimes fails on Windows
 --

 Key: FLINK-2377
 URL: https://issues.apache.org/jira/browse/FLINK-2377
 Project: Flink
  Issue Type: Bug
  Components: Tests
 Environment: Windows
Reporter: Gabor Gevay
Priority: Minor

 This is probably another file closing issue. (that is, Windows won't delete 
 open files, as opposed to Linux)
 I have encountered two concrete tests so far where this actually appears: 
 CsvOutputFormatITCase and CollectionSourceTest.



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