MAPREDUCE-6455. Unable to use surefire > 2.18. (Charlie Helin via kasha)

This commit is contained in:
Karthik Kambatla 2015-08-22 00:38:47 -07:00
parent 745d04be59
commit 61bf9cae6f
3 changed files with 12 additions and 3 deletions

View File

@ -988,8 +988,13 @@ private String substituteVars(String expr) {
} catch(SecurityException se) { } catch(SecurityException se) {
LOG.warn("Unexpected SecurityException in Configuration", se); LOG.warn("Unexpected SecurityException in Configuration", se);
} }
if (val == null) { if (val == null || val.isEmpty()) {
val = getRaw(var); String raw = getRaw(var);
if (raw != null) {
// if System.getProperty(var) returns an empty string, retain this
// value instead of return null
val = raw;
}
} }
if (val == null) { if (val == null) {
return eval; // return literal ${var}: var is unbound return eval; // return literal ${var}: var is unbound

View File

@ -275,6 +275,8 @@ Trunk (Unreleased)
MAPREDUCE-5801. Uber mode's log message is missing a vcore reason MAPREDUCE-5801. Uber mode's log message is missing a vcore reason
(Steven Wong via aw) (Steven Wong via aw)
MAPREDUCE-6455. Unable to use surefire > 2.18. (Charlie Helin via kasha)
Release 2.8.0 - UNRELEASED Release 2.8.0 - UNRELEASED
INCOMPATIBLE CHANGES INCOMPATIBLE CHANGES

View File

@ -181,7 +181,9 @@ static String obtainLogDirOwner(TaskAttemptID taskid) throws IOException {
} }
static String getBaseLogDir() { static String getBaseLogDir() {
return System.getProperty("hadoop.log.dir"); String logDir = System.getProperty("hadoop.log.dir");
// file is treating "" different from null {@see File#File(String, String)}
return logDir == null || logDir.isEmpty() ? null : logDir;
} }
static File getAttemptDir(TaskAttemptID taskid, boolean isCleanup) { static File getAttemptDir(TaskAttemptID taskid, boolean isCleanup) {