Shouldn't we do this only if `filePath != ""`? Otherwise we'll see that it doesn't exist and throw an exception below. You might want to format this so that the whole try block only happens if configFile is set or findResource returns a resource. Let me know if I'm missing something.. basically I'm worried about what happens if you don't set spark.metrics.conf and don't create a conf/metrics.properties either.