review comments

shardul-cr7 · shardul-cr7 · commit c908fd39eb22 · 2019-09-15T14:02:15.000+05:30
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/quotas/SpaceQuotaRefresherChore.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/quotas/SpaceQuotaRefresherChore.java
@@ -79,9 +79,10 @@ protected void chore() {
       // check whether Quota table is present or not.
       if (!quotaTablePresent && !checkQuotaTableExists()) {
         LOG.info("Quota table not found, skipping quota manager cache refresh.");
-        quotaTablePresent = true;
         return;
       }
+      // since quotaTable is present so setting the flag as true.
+      quotaTablePresent = true;
       if (LOG.isTraceEnabled()) {
         LOG.trace("Reading current quota snapshots from hbase:quota.");
       }

Original file line number	Diff line number	Diff line change
`@@ -79,9 +79,10 @@ protected void chore() {`
`79`	`79`	`// check whether Quota table is present or not.`
`80`	`80`	`if (!quotaTablePresent && !checkQuotaTableExists()) {`
`81`	`81`	`LOG.info("Quota table not found, skipping quota manager cache refresh.");`
`82`		`- quotaTablePresent = true;`
`83`	`82`	`return;`
`84`	`83`	`}`
	`84`	`+ // since quotaTable is present so setting the flag as true.`
	`85`	`+ quotaTablePresent = true;`
`85`	`86`	`if (LOG.isTraceEnabled()) {`
`86`	`87`	`LOG.trace("Reading current quota snapshots from hbase:quota.");`
`87`	`88`	`}`