KNOX-2944 - Honoring the impalad_specialization role configuration

apache · Jul 24, 2023 · 45ce7cd · 45ce7cd
1 parent 8c4db7a
commit 45ce7cd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/...g/apache/knox/gateway/topology/discovery/cm/model/impala/ImpalaServiceModelGenerator.java b/...g/apache/knox/gateway/topology/discovery/cm/model/impala/ImpalaServiceModelGenerator.java
@@ -100,6 +100,7 @@ public ServiceModel generateService(ApiService       service,
     ServiceModel model = createServiceModel(String.format(Locale.getDefault(), "%s://%s:%s/", scheme, hostname, port));
     model.addServiceProperty(SSL_ENABLED, getServiceConfigValue(serviceConfig, SSL_ENABLED));
     model.addRoleProperty(getRoleType(), HTTP_PORT, port);
+    model.addRoleProperty(getRoleType(), SPECIALIZATION, getRoleConfigValue(roleConfig, SPECIALIZATION));
 
     return model;
   }

diff --git a/...ache/knox/gateway/topology/discovery/cm/model/impala/ImpalaServiceModelGeneratorTest.java b/...ache/knox/gateway/topology/discovery/cm/model/impala/ImpalaServiceModelGeneratorTest.java
@@ -36,7 +36,7 @@ public void testServiceModelMetadata() {
     final Map<String, String> roleConfig = new HashMap<>();
     roleConfig.put(ImpalaServiceModelGenerator.HTTP_PORT, "1357");
 
-    validateServiceModel(createServiceModel(serviceConfig, roleConfig), serviceConfig, roleConfig);
+    validateServiceModel(createServiceModel(serviceConfig, roleConfig), serviceConfig, roleConfig, false);
   }
 
   @Override