You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by vj...@apache.org on 2023/05/15 20:55:30 UTC
[phoenix] branch 5.1 updated: PHOENIX-6951 Fix missing information on Index Repair Region and Time Taken to Repair (#1608)
This is an automated email from the ASF dual-hosted git repository.
vjasani pushed a commit to branch 5.1
in repository https://gitbox.apache.org/repos/asf/phoenix.git
The following commit(s) were added to refs/heads/5.1 by this push:
new 69c59076b0 PHOENIX-6951 Fix missing information on Index Repair Region and Time Taken to Repair (#1608)
69c59076b0 is described below
commit 69c59076b03851742b0a6cc04b32d8d0966d5ffe
Author: Jing Yu <40...@users.noreply.github.com>
AuthorDate: Mon May 15 13:54:00 2023 -0700
PHOENIX-6951 Fix missing information on Index Repair Region and Time Taken to Repair (#1608)
---
.../src/main/java/org/apache/phoenix/index/GlobalIndexChecker.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/index/GlobalIndexChecker.java b/phoenix-core/src/main/java/org/apache/phoenix/index/GlobalIndexChecker.java
index d9185cd478..24cb22c87d 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/index/GlobalIndexChecker.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/index/GlobalIndexChecker.java
@@ -98,7 +98,7 @@ public class GlobalIndexChecker extends BaseScannerRegionObserver implements Reg
private static final Logger LOG =
LoggerFactory.getLogger(GlobalIndexChecker.class);
private static final String REPAIR_LOGGING_PERCENT_ATTRIB = "phoenix.index.repair.logging.percent";
- private static final double DEFAULT_REPAIR_LOGGING_PERCENT = 1;
+ private static final double DEFAULT_REPAIR_LOGGING_PERCENT = 100;
private GlobalIndexCheckerSource metricsSource;
private CoprocessorEnvironment env;
@@ -605,8 +605,8 @@ public class GlobalIndexChecker extends BaseScannerRegionObserver implements Reg
metricsSource.updateIndexRepairTime(indexName,
EnvironmentEdgeManager.currentTimeMillis() - repairStart);
if (shouldLog()) {
- LOG.info(String.format("Index row repair on region {} took {} ms.",
- env.getRegionInfo().getRegionNameAsString(), repairTime));
+ LOG.info("Index row repair on region {} took {} ms.",
+ env.getRegionInfo().getRegionNameAsString(), repairTime);
}
} catch (IOException e) {
repairTime = EnvironmentEdgeManager.currentTimeMillis() - repairStart;