You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by am...@apache.org on 2018/08/01 21:16:06 UTC
asterixdb git commit: [NO ISSUE] Set logging level to WARN in
MaterializingPipelinedPartition
Repository: asterixdb
Updated Branches:
refs/heads/master 1027474db -> 708f3c37e
[NO ISSUE] Set logging level to WARN in MaterializingPipelinedPartition
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
Failures in MaterializingPipelinedPartition can be due to many reasons
such as a job cancellation. WARN log level seems to be adequate for
such failures keeping ERROR for more critical failures.
Change-Id: I45f475afae114dc71d2d89d9deaab7f3b3c5a0b8
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2832
Sonar-Qube: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: abdullah alamoudi <ba...@gmail.com>
Reviewed-by: Murtadha Hubail <mh...@apache.org>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Contrib: Jenkins <je...@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
Project: http://git-wip-us.apache.org/repos/asf/asterixdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/asterixdb/commit/708f3c37
Tree: http://git-wip-us.apache.org/repos/asf/asterixdb/tree/708f3c37
Diff: http://git-wip-us.apache.org/repos/asf/asterixdb/diff/708f3c37
Branch: refs/heads/master
Commit: 708f3c37e27bd41226c2e768ff640134b6d4bcac
Parents: 1027474
Author: Abdullah Alamoudi <ba...@gmail.com>
Authored: Wed Aug 1 11:35:28 2018 -0700
Committer: abdullah alamoudi <ba...@gmail.com>
Committed: Wed Aug 1 14:15:35 2018 -0700
----------------------------------------------------------------------
.../control/nc/partitions/MaterializingPipelinedPartition.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/asterixdb/blob/708f3c37/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/partitions/MaterializingPipelinedPartition.java
----------------------------------------------------------------------
diff --git a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/partitions/MaterializingPipelinedPartition.java b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/partitions/MaterializingPipelinedPartition.java
index 29c2ff2..37f33a9 100644
--- a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/partitions/MaterializingPipelinedPartition.java
+++ b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/partitions/MaterializingPipelinedPartition.java
@@ -89,7 +89,7 @@ public class MaterializingPipelinedPartition implements IFrameWriter, IPartition
Thread thread = Thread.currentThread();
setDataConsumerThread(thread); // Sets the data consumer thread to the current thread.
try {
- thread.setName(MaterializingPipelinedPartition.class.getName() + pid);
+ thread.setName(MaterializingPipelinedPartition.class.getName() + " " + pid);
FileReference fRefCopy;
synchronized (MaterializingPipelinedPartition.this) {
while (fRef == null && !eos && !failed) {
@@ -164,7 +164,7 @@ public class MaterializingPipelinedPartition implements IFrameWriter, IPartition
}
}
} catch (Exception e) {
- LOGGER.log(Level.ERROR, e.getMessage(), e);
+ LOGGER.warn("Failure writing to a frame", e);
} finally {
setDataConsumerThread(null); // Sets back the data consumer thread to null.
}