This is an automated email from the ASF dual-hosted git repository.
michaelsmith pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git
The following commit(s) were added to refs/heads/master by this push:
new f3ce97916 IMPALA-13426: Log Java debug sleeps at debug
f3ce97916 is described below
commit f3ce97916fe32c27d78d86467ee76589b20d7e6a
Author: Michael Smith <[email protected]>
AuthorDate: Mon Oct 7 13:30:25 2024 -0700
IMPALA-13426: Log Java debug sleeps at debug
Updates DebugUtils to log sleeps triggered by debug actions at debug
level, so they're visible in the dev and test environments. Debug
actions are opt-in, and it's important to identify if and when sleeps
happen to ensure tests are working correctly. This updates them to match
the backend, which already logs debug action sleeps at VLOG(1).
Change-Id: Ic5ffec41d62a8b7fb3730e5b3fb7b7dab9ef2972
Reviewed-on: http://gerrit.cloudera.org:8080/21904
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
fe/src/main/java/org/apache/impala/util/DebugUtils.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fe/src/main/java/org/apache/impala/util/DebugUtils.java
b/fe/src/main/java/org/apache/impala/util/DebugUtils.java
index 95bc7f390..28502d871 100644
--- a/fe/src/main/java/org/apache/impala/util/DebugUtils.java
+++ b/fe/src/main/java/org/apache/impala/util/DebugUtils.java
@@ -149,7 +149,7 @@ public class DebugUtils {
Preconditions.checkState(actionParams.size() == 2);
try {
int timeToSleepMs = Integer.parseInt(actionParams.get(1).trim());
- LOG.trace("Sleeping for {} msec to execute debug action {}",
+ LOG.debug("Sleeping for {} msec to execute debug action {}",
timeToSleepMs, label);
Thread.sleep(timeToSleepMs);
} catch (NumberFormatException ex) {
@@ -171,7 +171,7 @@ public class DebugUtils {
continue;
}
long timeToSleepMs = random.nextInt(maxTimeToSleepMs);
- LOG.trace("Sleeping for {} msec to execute debug action {}",
+ LOG.debug("Sleeping for {} msec to execute debug action {}",
timeToSleepMs, action);
Thread.sleep(timeToSleepMs);
} catch (NumberFormatException ex) {
