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 <jenk...@fulliautomatix.ics.uci.edu> Reviewed-by: abdullah alamoudi <bamou...@gmail.com> Reviewed-by: Murtadha Hubail <mhub...@apache.org> Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Contrib: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Integration-Tests: Jenkins <jenk...@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 <bamou...@gmail.com> Authored: Wed Aug 1 11:35:28 2018 -0700 Committer: abdullah alamoudi <bamou...@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. }