[
https://issues.apache.org/jira/browse/SPARK-6630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14396357#comment-14396357
]
Sean Owen commented on SPARK-6630:
----------------------------------
Hm, I don't think we can change this, since it would break binary
compatibility. Unless anyone has a bright-er idea, I think this is
unfortunately an optimization that can't be made now. Yeah we could make a
whole new method and all that but doesn't quite seem worth it.
> SparkConf.setIfMissing should only evaluate the assigned value if indeed
> missing
> --------------------------------------------------------------------------------
>
> Key: SPARK-6630
> URL: https://issues.apache.org/jira/browse/SPARK-6630
> Project: Spark
> Issue Type: Improvement
> Components: Spark Core
> Affects Versions: 1.3.0
> Reporter: Svend Vanderveken
> Priority: Minor
>
> The method setIfMissing() in SparkConf is currently systematically evaluating
> the right hand side of the assignment even if not used. This leads to
> unnecessary computation, like in the case of
> {code}
> conf.setIfMissing("spark.driver.host", Utils.localHostName())
> {code}
--
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
