diff options
author | Ryan Williams <ryan.blake.williams@gmail.com> | 2015-02-17 10:57:16 -0800 |
---|---|---|
committer | Patrick Wendell <patrick@databricks.com> | 2015-02-17 10:57:47 -0800 |
commit | 2bf2b56ef0c0fbb4cead030e44910453b2cbb6fd (patch) | |
tree | 907f8a5b20867c8bc7f744c5968f60535d913ace /.rat-excludes | |
parent | 4a581aa3f9144732cc0f6dab6e46f72035e072f4 (diff) | |
download | spark-2bf2b56ef0c0fbb4cead030e44910453b2cbb6fd.tar.gz spark-2bf2b56ef0c0fbb4cead030e44910453b2cbb6fd.tar.bz2 spark-2bf2b56ef0c0fbb4cead030e44910453b2cbb6fd.zip |
[SPARK-5778] throw if nonexistent metrics config file provided
previous behavior was to log an error; this is fine in the general
case where no `spark.metrics.conf` parameter was specified, in which
case a default `metrics.properties` is looked for, and the execption
logged and suppressed if it doesn't exist.
if the user has purposefully specified a metrics.conf file, however,
it makes more sense to show them an error when said file doesn't
exist.
Author: Ryan Williams <ryan.blake.williams@gmail.com>
Closes #4571 from ryan-williams/metrics and squashes the following commits:
5bccb14 [Ryan Williams] private-ize some MetricsConfig members
08ff998 [Ryan Williams] rename METRICS_CONF: DEFAULT_METRICS_CONF_FILENAME
f4d7fab [Ryan Williams] fix tests
ad24b0e [Ryan Williams] add "metrics.properties" to .rat-excludes
94e810b [Ryan Williams] throw if nonexistent Sink class is specified
31d2c30 [Ryan Williams] metrics code review feedback
56287db [Ryan Williams] throw if nonexistent metrics config file provided
(cherry picked from commit d8f69cf78862d13a48392a0b94388b8d403523da)
Signed-off-by: Patrick Wendell <patrick@databricks.com>
Diffstat (limited to '.rat-excludes')
-rw-r--r-- | .rat-excludes | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/.rat-excludes b/.rat-excludes index 769defbac1..e3c033145b 100644 --- a/.rat-excludes +++ b/.rat-excludes @@ -18,6 +18,7 @@ fairscheduler.xml.template spark-defaults.conf.template log4j.properties log4j.properties.template +metrics.properties metrics.properties.template slaves slaves.template |