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

Marcelo Vanzin commented on SPARK-8716:
---------------------------------------

bq. (1) It doesn't even work. Recently, each executor gets its own unique temp 
directory for security reasons.

I thought directories were per app, not per executor? (At least that was my 
goal when I changed that code.)

Not arguing against getting rid of the feature, just that this particular 
justification doesn't sound right.

> Remove executor shared cache feature
> ------------------------------------
>
>                 Key: SPARK-8716
>                 URL: https://issues.apache.org/jira/browse/SPARK-8716
>             Project: Spark
>          Issue Type: Improvement
>          Components: Spark Core
>    Affects Versions: 1.2.0
>            Reporter: Andrew Or
>            Assignee: Josh Rosen
>            Priority: Minor
>
> More specifically, this is the feature that is currently flagged by 
> `spark.files.useFetchCache`. There are several reasons why we should remove 
> it.
> (1) It doesn't even work. Recently, each executor gets its own unique temp 
> directory for security reasons.
> (2) There is no way to fix it. The constraints in (1) are fundamentally 
> opposed to sharing resources across executors.
> (3) It is very complex. The method Utils.fetchFile would be greatly 
> simplified without this feature that is not even used.
> (4) There are no tests for it and it is difficult to test.
> Note that we can't just revert the respective patches because they were 
> merged a long time ago.
> Related issues: SPARK-8130, SPARK-6313, SPARK-2713



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

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

Reply via email to