You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2021/07/27 02:44:46 UTC

[kylin] branch master updated: [KYLIN-5033] RealizationRegistry provider MUST always containing CubeManager

This is an automated email from the ASF dual-hosted git repository.

xxyu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/master by this push:
     new f803eab  [KYLIN-5033] RealizationRegistry provider MUST always containing CubeManager
f803eab is described below

commit f803eab1772bf91f04cf006d6b37d94b212b50e8
Author: yangjiang <ya...@ebay.com>
AuthorDate: Mon Jul 12 10:56:17 2021 +0800

    [KYLIN-5033] RealizationRegistry provider MUST always containing CubeManager
---
 .../org/apache/kylin/metadata/realization/RealizationRegistry.java    | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/core-metadata/src/main/java/org/apache/kylin/metadata/realization/RealizationRegistry.java b/core-metadata/src/main/java/org/apache/kylin/metadata/realization/RealizationRegistry.java
index 753d13e..6e3adc4 100644
--- a/core-metadata/src/main/java/org/apache/kylin/metadata/realization/RealizationRegistry.java
+++ b/core-metadata/src/main/java/org/apache/kylin/metadata/realization/RealizationRegistry.java
@@ -78,6 +78,10 @@ public class RealizationRegistry {
         if (providers.isEmpty())
             throw new IllegalArgumentException("Failed to find realization provider by url: " + config.getMetadataUrl());
 
+        // must have CubeManager in provides
+        if (!providers.containsKey(RealizationType.CUBE))
+            throw new IllegalArgumentException("Failed to init CubeManager by url: " + config.getMetadataUrl());
+
         logger.info("RealizationRegistry is " + providers);
     }