[kylin] branch master updated: KYLIN-3587 Including tableID in IllegalStateException messages

[shaofengshi]

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

shaofengshi 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 c8610e2  KYLIN-3587 Including tableID in IllegalStateException messages
c8610e2 is described below

commit c8610e21f4aaaa158ac400485bdab21ab59dc586
Author: Yichen Zhou <zhouy...@gmail.com>
AuthorDate: Tue Sep 25 18:14:51 2018 +0800

    KYLIN-3587 Including tableID in IllegalStateException messages
---
 .../java/org/apache/kylin/metadata/project/ProjectManager.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectManager.java
 
b/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectManager.java
index bad9773..77bad37 100644
--- 
a/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectManager.java
+++ 
b/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectManager.java
@@ -303,7 +303,7 @@ public class ProjectManager {
             for (String tableId : tableIdentities) {
                 TableDesc table = metaMgr.getTableDesc(tableId, projectName);
                 if (table == null) {
-                    throw new IllegalStateException("Cannot find table '" + 
table + "' in metadata manager");
+                    throw new IllegalStateException("Cannot find table '" + 
tableId + "' in metadata manager");
                 }
                 projectInstance.addTable(table.getIdentity());
             }
@@ -312,13 +312,13 @@ public class ProjectManager {
         }
     }
 
-    public void removeTableDescFromProject(String tableIdentities, String 
projectName) throws IOException {
+    public void removeTableDescFromProject(String tableId, String projectName) 
throws IOException {
         try (AutoLock lock = prjMapLock.lockForWrite()) {
             TableMetadataManager metaMgr = getTableManager();
             ProjectInstance projectInstance = getProject(projectName);
-            TableDesc table = metaMgr.getTableDesc(tableIdentities, 
projectName);
+            TableDesc table = metaMgr.getTableDesc(tableId, projectName);
             if (table == null) {
-                throw new IllegalStateException("Cannot find table '" + table 
+ "' in metadata manager");
+                throw new IllegalStateException("Cannot find table '" + 
tableId + "' in metadata manager");
             }
 
             projectInstance.removeTable(table.getIdentity());