[phoenix] branch 5.1 updated: PHOENIX-6951 Fix missing information on Index Repair Region and Time Taken to Repair (#1608)

[vjasani]

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 <40223124+jing...@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;