[kylin] branch master updated: KYLIN-3721 modify sourceCacheKey for RDBMS

[sh...@apache.org]

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 9fb5b46  KYLIN-3721 modify sourceCacheKey for RDBMS
9fb5b46 is described below

commit 9fb5b4657edcb58d370bd64e5b011172188c4c37
Author: woyumen4597 <wo...@gmail.com>
AuthorDate: Sun Dec 16 11:29:29 2018 +0800

    KYLIN-3721 modify sourceCacheKey for RDBMS
---
 core-metadata/src/main/java/org/apache/kylin/source/SourceManager.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/core-metadata/src/main/java/org/apache/kylin/source/SourceManager.java b/core-metadata/src/main/java/org/apache/kylin/source/SourceManager.java
index 03559bc..c0bd322 100644
--- a/core-metadata/src/main/java/org/apache/kylin/source/SourceManager.java
+++ b/core-metadata/src/main/java/org/apache/kylin/source/SourceManager.java
@@ -103,9 +103,10 @@ public class SourceManager {
         builder.append(config.getJdbcSourceConnectionUrl()).append('|');
         builder.append(config.getJdbcSourceDriver()).append('|');
         builder.append(config.getJdbcSourceUser()).append('|');
+        builder.append(config.getJdbcSourcePass()).append('|'); // In case password is wrong at the first time
         builder.append(config.getJdbcSourceFieldDelimiter()).append('|');
         builder.append(config.getJdbcSourceDialect()).append('|');
-        return builder.toString(); // jdbc password not needed, because url+user should be identical.
+        return builder.toString();
     }
 
     private ISource createSource(ISourceAware aware) {