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 2022/12/05 10:21:13 UTC

[kylin] 21/22: KYLIN-5325 Fix the number type conversion problem caused by writing JSON files

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 c35921431951f41bb5c4a645835d0fd10747c786
Author: Guoliang Sun <gu...@kyligence.io>
AuthorDate: Tue Oct 18 11:02:37 2022 +0800

    KYLIN-5325 Fix the number type conversion problem caused by writing JSON files
---
 .../src/main/java/org/apache/kylin/metrics/HdfsCapacityMetrics.java   | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/core-metadata/src/main/java/org/apache/kylin/metrics/HdfsCapacityMetrics.java b/src/core-metadata/src/main/java/org/apache/kylin/metrics/HdfsCapacityMetrics.java
index 9b114779ea..236fb5c556 100644
--- a/src/core-metadata/src/main/java/org/apache/kylin/metrics/HdfsCapacityMetrics.java
+++ b/src/core-metadata/src/main/java/org/apache/kylin/metrics/HdfsCapacityMetrics.java
@@ -155,7 +155,9 @@ public class HdfsCapacityMetrics {
      */
     public static Long getHdfsCapacityByProject(String project) {
         if (hdfsMetricsPeriodicCalculationEnabled) {
-            return workingDirCapacity.getOrDefault(project, 0L);
+            // Writing numbers in JSON may be read as integer
+            Object orDefault = workingDirCapacity.getOrDefault(project, 0L);
+            return Long.parseLong(orDefault.toString());
         }
         return -1L;
     }