ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From srima...@apache.org
Subject git commit: AMBARI-8151. Slider View: /slider/apptypes returns unsecured configs when cluster is secured and ambari-server not restarted (srimanth)
Date Tue, 04 Nov 2014 23:04:19 GMT
Repository: ambari
Updated Branches:
  refs/heads/trunk 0e5804f4c -> 804fab44b


AMBARI-8151. Slider View: /slider/apptypes returns unsecured configs when cluster is secured
and ambari-server not restarted (srimanth)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/804fab44
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/804fab44
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/804fab44

Branch: refs/heads/trunk
Commit: 804fab44b59ecb20096f0a02725e423e65e8b939
Parents: 0e5804f
Author: Srimanth Gunturi <sgunturi@hortonworks.com>
Authored: Tue Nov 4 14:55:10 2014 -0800
Committer: Srimanth Gunturi <sgunturi@hortonworks.com>
Committed: Tue Nov 4 15:04:07 2014 -0800

----------------------------------------------------------------------
 .../view/slider/SliderAppsViewControllerImpl.java      | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/804fab44/contrib/views/slider/src/main/java/org/apache/ambari/view/slider/SliderAppsViewControllerImpl.java
----------------------------------------------------------------------
diff --git a/contrib/views/slider/src/main/java/org/apache/ambari/view/slider/SliderAppsViewControllerImpl.java
b/contrib/views/slider/src/main/java/org/apache/ambari/view/slider/SliderAppsViewControllerImpl.java
index 0370372..1a52efa 100644
--- a/contrib/views/slider/src/main/java/org/apache/ambari/view/slider/SliderAppsViewControllerImpl.java
+++ b/contrib/views/slider/src/main/java/org/apache/ambari/view/slider/SliderAppsViewControllerImpl.java
@@ -944,14 +944,13 @@ public class SliderAppsViewControllerImpl implements SliderAppsViewController
{
 
   @Override
   public List<SliderAppType> getSliderAppTypes(Set<String> properties) {
-    Map<String, String> hadoopConfigs = getHadoopConfigs();
-    if(hadoopConfigs==null || hadoopConfigs.isEmpty()) {
-      try {
-        // Need to determine security enablement before loading app types
-        getViewStatus();
-      } catch (Throwable t) {
-      }
+    try {
+      // Need to determine security enablement before loading app types
+      getViewStatus();
+    } catch (Throwable t) {
+      logger.warn("Unable to determine if cluster is secured when loading app-types", t);
     }
+    Map<String, String> hadoopConfigs = getHadoopConfigs();
     final boolean securityEnabled = Boolean.valueOf(hadoopConfigs.get("security_enabled"));
     if (appTypes == null) {
       appTypes = loadAppTypes();


Mime
View raw message