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:16 -0800 |
| commit | d8f69cf78862d13a48392a0b94388b8d403523da (patch) | |
| tree | f95761f507fb6fdc16b73e4c5f07a52e8c82d17f /.rat-excludes | |
| parent | d8adefefcc2a4af32295440ed1d4917a6968f017 (diff) | |
| download | spark-d8f69cf78862d13a48392a0b94388b8d403523da.tar.gz spark-d8f69cf78862d13a48392a0b94388b8d403523da.tar.bz2 spark-d8f69cf78862d13a48392a0b94388b8d403523da.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
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 a788e8273d..8c61e67a0c 100644 --- a/.rat-excludes +++ b/.rat-excludes @@ -19,6 +19,7 @@ fairscheduler.xml.template spark-defaults.conf.template log4j.properties log4j.properties.template +metrics.properties metrics.properties.template slaves slaves.template |