[jira] [Commented] (SPARK-33384) Delete temporary file when cancelling writing to final path even underlying stream throwing error

2020-11-08 Thread Apache Spark (Jira) 


[ 
https://issues.apache.org/jira/browse/SPARK-33384?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17228270#comment-17228270
 ] 

Apache Spark commented on SPARK-33384:
--

User 'viirya' has created a pull request for this issue:
https://github.com/apache/spark/pull/30290

> Delete temporary file when cancelling writing to final path even underlying 
> stream throwing error
> -
>
> Key: SPARK-33384
> URL: https://issues.apache.org/jira/browse/SPARK-33384
> Project: Spark
>  Issue Type: Improvement
>  Components: Structured Streaming
>Affects Versions: 3.1.0
>Reporter: L. C. Hsieh
>Assignee: L. C. Hsieh
>Priority: Minor
>
> In {{RenameBasedFSDataOutputStream.cancel}}, we do two things: closing 
> underlying stream and delete temporary file, in a single try/catch block. 
> Closing {{OutputStream}} could possibly throw {{IOException}} so we possibly 
> missing deleting temporary file.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

[jira] [Commented] (SPARK-33384) Delete temporary file when cancelling writing to final path even underlying stream throwing error

2020-11-08 Thread Apache Spark (Jira) 


[ 
https://issues.apache.org/jira/browse/SPARK-33384?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17228269#comment-17228269
 ] 

Apache Spark commented on SPARK-33384:
--

User 'viirya' has created a pull request for this issue:
https://github.com/apache/spark/pull/30290

> Delete temporary file when cancelling writing to final path even underlying 
> stream throwing error
> -
>
> Key: SPARK-33384
> URL: https://issues.apache.org/jira/browse/SPARK-33384
> Project: Spark
>  Issue Type: Improvement
>  Components: Structured Streaming
>Affects Versions: 3.1.0
>Reporter: L. C. Hsieh
>Assignee: L. C. Hsieh
>Priority: Minor
>
> In {{RenameBasedFSDataOutputStream.cancel}}, we do two things: closing 
> underlying stream and delete temporary file, in a single try/catch block. 
> Closing {{OutputStream}} could possibly throw {{IOException}} so we possibly 
> missing deleting temporary file.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org