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 2023/02/27 08:01:10 UTC

[kylin] 30/34: [DIRTY] fix unstable test

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

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

commit 8f7f5eb5ff8b93f50a13ae28ce27673b21808a5b
Author: qianhao.zhou <z....@gmail.com>
AuthorDate: Thu Jan 5 18:48:29 2023 +0800

    [DIRTY] fix unstable test
    
    Co-authored-by: qhzhou <qi...@kyligence.io>
---
 .../src/main/java/org/apache/kylin/helper/MetadataToolHelper.java      | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/common-service/src/main/java/org/apache/kylin/helper/MetadataToolHelper.java b/src/common-service/src/main/java/org/apache/kylin/helper/MetadataToolHelper.java
index 2804bdc4c3..7ca9d6fd86 100644
--- a/src/common-service/src/main/java/org/apache/kylin/helper/MetadataToolHelper.java
+++ b/src/common-service/src/main/java/org/apache/kylin/helper/MetadataToolHelper.java
@@ -135,7 +135,8 @@ public class MetadataToolHelper {
                     + "_backup";
         }
         String backupPath = StringUtils.appendIfMissing(path, "/") + folder;
-        logger.info("The metadata backup path is {}}", backupPath);
+        //FIXME should replace printf with Logger while Logger MUST print this message to console, because test depends on it
+        System.out.printf(Locale.ROOT, "The metadata backup path is %s.%n", backupPath);
         val backupMetadataUrl = getMetadataUrl(backupPath, compress, kylinConfig);
         val backupConfig = KylinConfig.createKylinConfig(kylinConfig);
         backupConfig.setMetadataUrl(backupMetadataUrl);