You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by dm...@apache.org on 2017/04/21 13:10:16 UTC
[2/2] ambari git commit: AMBARI-20811. Trying to upgrade HDF 2.0.2.0
to HDF 3.0.0.0 fails with NPE (dlysnichenko)
AMBARI-20811. Trying to upgrade HDF 2.0.2.0 to HDF 3.0.0.0 fails with NPE (dlysnichenko)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/04f31104
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/04f31104
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/04f31104
Branch: refs/heads/trunk
Commit: 04f31104bd481426823204468ea02f0fe8c33c48
Parents: a94abb1
Author: Lisnichenko Dmitro <dl...@hortonworks.com>
Authored: Fri Apr 21 16:09:01 2017 +0300
Committer: Lisnichenko Dmitro <dl...@hortonworks.com>
Committed: Fri Apr 21 16:09:47 2017 +0300
----------------------------------------------------------------------
.../java/org/apache/ambari/server/stack/StackDirectory.java | 4 ++++
.../java/org/apache/ambari/server/stack/StackManagerTest.java | 6 +++++-
2 files changed, 9 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/04f31104/ambari-server/src/main/java/org/apache/ambari/server/stack/StackDirectory.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/stack/StackDirectory.java b/ambari-server/src/main/java/org/apache/ambari/server/stack/StackDirectory.java
index d0950a7..5b44112 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/stack/StackDirectory.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/stack/StackDirectory.java
@@ -20,6 +20,7 @@ package org.apache.ambari.server.stack;
import java.io.File;
import java.io.IOException;
+import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
import java.util.HashMap;
@@ -437,6 +438,9 @@ public class StackDirectory extends StackDefinitionDirectory {
if (configUpgradePack != null) {
this.configUpgradePack = configUpgradePack;
} else {
+ ConfigUpgradePack emptyConfigUpgradePack = new ConfigUpgradePack();
+ emptyConfigUpgradePack.services = new ArrayList<>();
+ this.configUpgradePack = emptyConfigUpgradePack;
LOG.info("Stack '{}' doesn't contain config upgrade pack file", getPath());
}
}
http://git-wip-us.apache.org/repos/asf/ambari/blob/04f31104/ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java
----------------------------------------------------------------------
diff --git a/ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java b/ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java
index 8c7ab9d..d8c02dc 100644
--- a/ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java
+++ b/ambari-server/src/test/java/org/apache/ambari/server/stack/StackManagerTest.java
@@ -58,6 +58,7 @@ import org.apache.ambari.server.state.PropertyInfo;
import org.apache.ambari.server.state.ServiceInfo;
import org.apache.ambari.server.state.ServiceOsSpecific;
import org.apache.ambari.server.state.StackInfo;
+import org.apache.ambari.server.state.stack.ConfigUpgradePack;
import org.apache.ambari.server.state.stack.MetricDefinition;
import org.apache.ambari.server.state.stack.OsFamily;
import org.apache.ambari.server.state.stack.UpgradePack;
@@ -728,7 +729,7 @@ public class StackManagerTest {
}
/**
- * Tests that {@link UpgradePack} instances are correctly initialized
+ * Tests that {@link UpgradePack} and {@link ConfigUpgradePack} instances are correctly initialized
* post-unmarshalling.
*
* @throws Exception
@@ -745,6 +746,9 @@ public class StackManagerTest {
// reference equality (make sure it's the same list)
assertTrue(upgradePack.getTasks() == upgradePack.getTasks());
}
+ ConfigUpgradePack configUpgradePack = stack.getConfigUpgradePack();
+ assertNotNull(configUpgradePack);
+ assertNotNull(configUpgradePack.services);
}
@Test