You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by ma...@apache.org on 2016/04/12 10:04:40 UTC
kylin git commit: minor bug fix in removing ext filter from project
Repository: kylin
Updated Branches:
refs/heads/yang-m1 c1af3c015 -> 71113643f
minor bug fix in removing ext filter from project
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/71113643
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/71113643
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/71113643
Branch: refs/heads/yang-m1
Commit: 71113643fc3d3137e0baf3edbba513ee0fd47820
Parents: c1af3c0
Author: Hongbin Ma <ma...@apache.org>
Authored: Tue Apr 12 16:04:32 2016 +0800
Committer: Hongbin Ma <ma...@apache.org>
Committed: Tue Apr 12 16:04:32 2016 +0800
----------------------------------------------------------------------
.../src/main/java/org/apache/kylin/metadata/MetadataManager.java | 3 ---
.../java/org/apache/kylin/metadata/project/ProjectInstance.java | 4 ++--
2 files changed, 2 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/71113643/core-metadata/src/main/java/org/apache/kylin/metadata/MetadataManager.java
----------------------------------------------------------------------
diff --git a/core-metadata/src/main/java/org/apache/kylin/metadata/MetadataManager.java b/core-metadata/src/main/java/org/apache/kylin/metadata/MetadataManager.java
index 9ed1f13..26ee9c4 100644
--- a/core-metadata/src/main/java/org/apache/kylin/metadata/MetadataManager.java
+++ b/core-metadata/src/main/java/org/apache/kylin/metadata/MetadataManager.java
@@ -228,9 +228,6 @@ public class MetadataManager {
}
public void removeExternalFilter(String name) throws IOException {
- if(name !=null ){
- name = name.toLowerCase();
- }
String path = ExternalFilterDesc.concatResourcePath(name);
getStore().deleteResource(path);
extFilterMap.remove(name);
http://git-wip-us.apache.org/repos/asf/kylin/blob/71113643/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java
----------------------------------------------------------------------
diff --git a/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java b/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java
index 9567da3..b488d07 100644
--- a/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java
+++ b/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java
@@ -212,7 +212,7 @@ public class ProjectInstance extends RootPersistentEntity {
}
public void removeExtFilter(String filterName) {
- extFilters.remove(filterName.toUpperCase());
+ extFilters.remove(filterName);
}
public int getTablesCount() {
@@ -228,7 +228,7 @@ public class ProjectInstance extends RootPersistentEntity {
}
public void addExtFilter(String extFilterName){
- this.getExtFilters().add(extFilterName.toLowerCase());
+ this.getExtFilters().add(extFilterName);
}
public Set<String> getExtFilters(){