diff options
| author | Sean Owen <sowen@cloudera.com> | 2017-04-09 08:44:02 +0100 |
|---|---|---|
| committer | Sean Owen <sowen@cloudera.com> | 2017-04-09 08:44:02 +0100 |
| commit | 1f0de3c1c85a41eadc7c4131bdc948405f340099 (patch) | |
| tree | ce81e88b98844a940ddc1027cbffdcbe1d872ba4 /core/src/main/scala | |
| parent | 34fc48fb5976ede00f3f6d8c4d3eec979e4f4d7f (diff) | |
| download | spark-1f0de3c1c85a41eadc7c4131bdc948405f340099.tar.gz spark-1f0de3c1c85a41eadc7c4131bdc948405f340099.tar.bz2 spark-1f0de3c1c85a41eadc7c4131bdc948405f340099.zip | |
[SPARK-19991][CORE][YARN] FileSegmentManagedBuffer performance improvement
## What changes were proposed in this pull request?
Avoid `NoSuchElementException` every time `ConfigProvider.get(val, default)` falls back to default. This apparently causes non-trivial overhead in at least one path, and can easily be avoided.
See https://github.com/apache/spark/pull/17329
## How was this patch tested?
Existing tests
Author: Sean Owen <sowen@cloudera.com>
Closes #17567 from srowen/SPARK-19991.
Diffstat (limited to 'core/src/main/scala')
| -rw-r--r-- | core/src/main/scala/org/apache/spark/network/netty/SparkTransportConf.scala | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/main/scala/org/apache/spark/network/netty/SparkTransportConf.scala b/core/src/main/scala/org/apache/spark/network/netty/SparkTransportConf.scala index df520f804b..25f7bcb980 100644 --- a/core/src/main/scala/org/apache/spark/network/netty/SparkTransportConf.scala +++ b/core/src/main/scala/org/apache/spark/network/netty/SparkTransportConf.scala @@ -60,7 +60,7 @@ object SparkTransportConf { new TransportConf(module, new ConfigProvider { override def get(name: String): String = conf.get(name) - + override def get(name: String, defaultValue: String): String = conf.get(name, defaultValue) override def getAll(): java.lang.Iterable[java.util.Map.Entry[String, String]] = { conf.getAll.toMap.asJava.entrySet() } |