You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by jl...@apache.org on 2019/01/08 18:35:02 UTC
[incubator-pinot] branch fix-log-error-message created (now fe3cedf)
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a change to branch fix-log-error-message
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.
at fe3cedf Fix error messages
This branch includes the following new commits:
new fe3cedf Fix error messages
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[incubator-pinot] 01/01: Fix error messages
Posted by jl...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
jlli pushed a commit to branch fix-log-error-message
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
commit fe3cedfdf01d1306a72e0f753afa2f2ee20158f8
Author: Jack Li(Analytics Engineering) <jl...@jlli-mn1.linkedin.biz>
AuthorDate: Tue Jan 8 10:34:42 2019 -0800
Fix error messages
---
.../java/com/linkedin/pinot/controller/helix/SegmentStatusChecker.java | 2 +-
.../pinot/controller/helix/core/retention/RetentionManager.java | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/pinot-controller/src/main/java/com/linkedin/pinot/controller/helix/SegmentStatusChecker.java b/pinot-controller/src/main/java/com/linkedin/pinot/controller/helix/SegmentStatusChecker.java
index d8fdbb4..c62d74b 100644
--- a/pinot-controller/src/main/java/com/linkedin/pinot/controller/helix/SegmentStatusChecker.java
+++ b/pinot-controller/src/main/java/com/linkedin/pinot/controller/helix/SegmentStatusChecker.java
@@ -240,7 +240,7 @@ public class SegmentStatusChecker extends ControllerPeriodicTask {
nReplicasIdealMax);
}
} catch (Exception e) {
- LOGGER.warn("Caught exception while updating segment status for table {}", e, tableNameWithType);
+ LOGGER.warn("Caught exception while updating segment status for table {}", tableNameWithType, e);
// Remove the metric for this table
resetTableMetrics(tableNameWithType);
diff --git a/pinot-controller/src/main/java/com/linkedin/pinot/controller/helix/core/retention/RetentionManager.java b/pinot-controller/src/main/java/com/linkedin/pinot/controller/helix/core/retention/RetentionManager.java
index 134c4ee..426ca55 100644
--- a/pinot-controller/src/main/java/com/linkedin/pinot/controller/helix/core/retention/RetentionManager.java
+++ b/pinot-controller/src/main/java/com/linkedin/pinot/controller/helix/core/retention/RetentionManager.java
@@ -74,7 +74,7 @@ public class RetentionManager extends ControllerPeriodicTask {
LOGGER.info("Start managing retention for table: {}", tableNameWithType);
manageRetentionForTable(tableNameWithType);
} catch (Exception e) {
- LOGGER.error("Caught exception while managing retention for all tables", e);
+ LOGGER.error("Caught exception while managing retention for table: {}", tableNameWithType, e);
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org