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 /core/src/test/resources/test_metrics_system.properties | |
| 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 'core/src/test/resources/test_metrics_system.properties')
| -rw-r--r-- | core/src/test/resources/test_metrics_system.properties | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/core/src/test/resources/test_metrics_system.properties b/core/src/test/resources/test_metrics_system.properties index 35d0bd3b8d..4e8b846569 100644 --- a/core/src/test/resources/test_metrics_system.properties +++ b/core/src/test/resources/test_metrics_system.properties @@ -18,7 +18,5 @@ *.sink.console.period = 10 *.sink.console.unit = seconds test.sink.console.class = org.apache.spark.metrics.sink.ConsoleSink -test.sink.dummy.class = org.apache.spark.metrics.sink.DummySink -test.source.dummy.class = org.apache.spark.metrics.source.DummySource test.sink.console.period = 20 test.sink.console.unit = minutes |