You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by mb...@apache.org on 2023/12/05 03:37:33 UTC
(asterixdb) 01/07: [NO ISSUE] Use getClass().getName() instead of getClass().getSimpleName()
This is an automated email from the ASF dual-hosted git repository.
mblow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git
commit 78ebed0d37f18308bcd93ca678c9bc327f10fe99
Author: Ali Alsuliman <al...@gmail.com>
AuthorDate: Thu Nov 23 22:45:41 2023 -0800
[NO ISSUE] Use getClass().getName() instead of getClass().getSimpleName()
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
Use getClass().getName() instead of getClass().getSimpleName()
when logging closable failures to include anonymous classes.
Change-Id: I5e6e056000f7162e00e25a3b071c6ba1156cff6b
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17983
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Michael Blow <mb...@apache.org>
---
.../src/main/java/org/apache/hyracks/api/util/CleanupUtils.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/CleanupUtils.java b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/CleanupUtils.java
index 220311eea4..a29a04aee6 100644
--- a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/CleanupUtils.java
+++ b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/CleanupUtils.java
@@ -67,7 +67,7 @@ public class CleanupUtils {
} catch (Throwable th) { // NOSONAR Will be suppressed
try {
LOGGER.log(ExceptionUtils.causedByInterrupt(th) ? Level.DEBUG : Level.WARN,
- "Failure closing a closeable resource of class {}", writer.getClass().getSimpleName(), th);
+ "Failure closing a closeable resource of class {}", writer.getClass().getName(), th);
} catch (Throwable loggingFailure) { // NOSONAR: Ignore catching Throwable
// NOSONAR: Ignore logging failure
}
@@ -92,7 +92,7 @@ public class CleanupUtils {
} catch (Throwable th) { // NOSONAR Will be suppressed
try {
LOGGER.log(ExceptionUtils.causedByInterrupt(th) ? Level.DEBUG : Level.WARN,
- "Failure failing " + writer.getClass().getSimpleName(), th);
+ "Failure failing " + writer.getClass().getName(), th);
} catch (Throwable loggingFailure) { // NOSONAR: Ignore catching Throwable
// NOSONAR ignore logging failure
}
@@ -119,7 +119,7 @@ public class CleanupUtils {
} catch (Throwable th) { // NOSONAR Will be suppressed
try {
LOGGER.log(ExceptionUtils.causedByInterrupt(th) ? Level.DEBUG : Level.WARN,
- "Failure closing a closeable resource", th);
+ "Failure closing a closeable resource {}", closable.getClass().getName(), th);
} catch (Throwable loggingFailure) { // NOSONAR: Ignore catching Throwable
// NOSONAR ignore logging failure
}