[kylin] branch main updated: minor,log print format

[xxyu]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new 63f9ac6bcd minor,log print format
63f9ac6bcd is described below

commit 63f9ac6bcd0db005f10935d88747d39fc0819ab7
Author: zhaoliu4 <zhaol...@iflytek.com>
AuthorDate: Fri Oct 14 10:25:30 2022 +0800

    minor,log print format
---
 .../src/main/java/org/apache/kylin/rest/job/MetadataCleanupJob.java | 2 +-
 .../kylin/rest/signature/FactTableRealizationSetCalculator.java     | 6 +++---
 tool/src/main/java/org/apache/kylin/tool/ExtendCubeToHybridCLI.java | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git 
a/server-base/src/main/java/org/apache/kylin/rest/job/MetadataCleanupJob.java 
b/server-base/src/main/java/org/apache/kylin/rest/job/MetadataCleanupJob.java
index 3ae2fb197d..e72c5b5b7a 100644
--- 
a/server-base/src/main/java/org/apache/kylin/rest/job/MetadataCleanupJob.java
+++ 
b/server-base/src/main/java/org/apache/kylin/rest/job/MetadataCleanupJob.java
@@ -162,7 +162,7 @@ public class MetadataCleanupJob {
         try {
             timestamp = store.getResourceTimestamp(resPath);
         } catch (IOException e) {
-            logger.warn("Failed to get resource timestamp from remote resource 
store, details:{}", e);
+            logger.warn("Failed to get resource timestamp from remote resource 
store, details:", e);
         }
         return timestamp;
     }
diff --git 
a/server-base/src/main/java/org/apache/kylin/rest/signature/FactTableRealizationSetCalculator.java
 
b/server-base/src/main/java/org/apache/kylin/rest/signature/FactTableRealizationSetCalculator.java
index f998fdd085..bb46b068e9 100644
--- 
a/server-base/src/main/java/org/apache/kylin/rest/signature/FactTableRealizationSetCalculator.java
+++ 
b/server-base/src/main/java/org/apache/kylin/rest/signature/FactTableRealizationSetCalculator.java
@@ -75,17 +75,17 @@ public class FactTableRealizationSetCalculator extends 
RealizationSetCalculator
 
     private String getRootFactTableForRealization(IRealization realization) {
         if (realization == null) {
-            logger.warn("Cannot find realization %s", realization);
+            logger.warn("Cannot find realization {}", realization);
             return null;
         }
         DataModelDesc model = realization.getModel();
         if (model == null) {
-            logger.warn("The model for realization %s is null", 
realization.getName());
+            logger.warn("The model for realization {} is null", 
realization.getName());
             return null;
         }
         TableRef rootFactTable = model.getRootFactTable();
         if (rootFactTable == null) {
-            logger.warn("The root table for model %s is null", 
model.getName());
+            logger.warn("The root table for model {} is null", 
model.getName());
             return null;
         }
         return rootFactTable.getTableIdentity();
diff --git 
a/tool/src/main/java/org/apache/kylin/tool/ExtendCubeToHybridCLI.java 
b/tool/src/main/java/org/apache/kylin/tool/ExtendCubeToHybridCLI.java
index c45428610a..dcf83a6995 100644
--- a/tool/src/main/java/org/apache/kylin/tool/ExtendCubeToHybridCLI.java
+++ b/tool/src/main/java/org/apache/kylin/tool/ExtendCubeToHybridCLI.java
@@ -101,7 +101,7 @@ public class ExtendCubeToHybridCLI {
             logger.info("Job Finished.");
         } catch (Exception e) {
             e.printStackTrace();
-            logger.error("Job Aborted.", e.getMessage());
+            logger.error("Job Aborted. msg:{}", e.getMessage());
         }
     }