You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by le...@apache.org on 2020/04/19 13:33:14 UTC
[incubator-hudi] branch master updated: [HUDI-800] Fix Metrics
getReporter().close() throws NPE. (#1529)
This is an automated email from the ASF dual-hosted git repository.
leesf pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git
The following commit(s) were added to refs/heads/master by this push:
new 09fd6f6 [HUDI-800] Fix Metrics getReporter().close() throws NPE. (#1529)
09fd6f6 is described below
commit 09fd6f64c527e6a822c4e17dc4e61b8fdee28189
Author: lw0090 <lw...@gmail.com>
AuthorDate: Sun Apr 19 21:33:07 2020 +0800
[HUDI-800] Fix Metrics getReporter().close() throws NPE. (#1529)
---
hudi-client/src/main/java/org/apache/hudi/metrics/Metrics.java | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/hudi-client/src/main/java/org/apache/hudi/metrics/Metrics.java b/hudi-client/src/main/java/org/apache/hudi/metrics/Metrics.java
index b62279e..5980911 100644
--- a/hudi-client/src/main/java/org/apache/hudi/metrics/Metrics.java
+++ b/hudi-client/src/main/java/org/apache/hudi/metrics/Metrics.java
@@ -52,9 +52,11 @@ public class Metrics {
Runtime.getRuntime().addShutdownHook(new Thread(() -> {
try {
reporter.report();
- getReporter().close();
+ if (getReporter() != null) {
+ getReporter().close();
+ }
} catch (Exception e) {
- e.printStackTrace();
+ LOG.warn("Error while closing reporter", e);
}
}));
}