[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)

[morningman]

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 <55968745+caicong...@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-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org