You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2020/07/26 12:14:13 UTC

[incubator-doris] branch master updated: [Bug][UpdateDataQuota] Skip update used data quota for information_schema db and fix bug for wrong time interval for UpdateDbUsedDataQuotaDaemon (#4175)

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

morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 911eb04  [Bug][UpdateDataQuota] Skip update used data quota for information_schema db and fix bug for wrong time interval for  UpdateDbUsedDataQuotaDaemon (#4175)
911eb04 is described below

commit 911eb04594e893af0b4394eb405105b783775b2a
Author: caiconghui <55...@users.noreply.github.com>
AuthorDate: Sun Jul 26 07:14:03 2020 -0500

    [Bug][UpdateDataQuota] Skip update used data quota for information_schema db and fix bug for wrong time interval for  UpdateDbUsedDataQuotaDaemon (#4175)
    
    This PR is to skip update used data quota for information_schema db,
    and fix bug for wrong time interval for UpdateDbUsedDataQuotaDaemon.
---
 .../org/apache/doris/transaction/UpdateDbUsedDataQuotaDaemon.java  | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/transaction/UpdateDbUsedDataQuotaDaemon.java b/fe/fe-core/src/main/java/org/apache/doris/transaction/UpdateDbUsedDataQuotaDaemon.java
index e8336a1..ffe073d 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/transaction/UpdateDbUsedDataQuotaDaemon.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/transaction/UpdateDbUsedDataQuotaDaemon.java
@@ -31,7 +31,7 @@ public class UpdateDbUsedDataQuotaDaemon extends MasterDaemon {
     private static final Logger LOG = LogManager.getLogger(UpdateDbUsedDataQuotaDaemon.class);
 
     public UpdateDbUsedDataQuotaDaemon() {
-        super("UPDATE_DB_USED_QUOTA", Config.db_used_data_quota_update_interval_secs);
+        super("UpdateDbUsedDataQuota", Config.db_used_data_quota_update_interval_secs * 1000);
     }
 
     @Override
@@ -49,6 +49,9 @@ public class UpdateDbUsedDataQuotaDaemon extends MasterDaemon {
                 LOG.warn("Database [" + dbId + "] doese not exist, skip to update database used data quota");
                 continue;
             }
+            if (db.isInfoSchemaDb()) {
+                continue;
+            }
             try {
                 long usedDataQuotaBytes = db.getUsedDataQuotaWithLock();
                 globalTransactionMgr.updateDatabaseUsedQuotaData(dbId, usedDataQuotaBytes);
@@ -56,7 +59,7 @@ public class UpdateDbUsedDataQuotaDaemon extends MasterDaemon {
                     LOG.debug("Update database[{}] used data quota bytes : {}.", db.getFullName(), usedDataQuotaBytes);
                 }
             } catch (AnalysisException e) {
-                LOG.warn("Update database[" + dbId + "] used data quota failed", e);
+                LOG.warn("Update database[" + db.getFullName() + "] used data quota failed", e);
             }
         }
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org