Michael Blow has submitted this change and it was merged. Change subject: [NO ISSUE] Misc logging cleanup ......................................................................
[NO ISSUE] Misc logging cleanup Change-Id: I2b951e11a9716eb11832d18eba57b124faf146b1 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2295 Sonar-Qube: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Contrib: Jenkins <[email protected]> Reviewed-by: Murtadha Hubail <[email protected]> --- M hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/meta/AlgebricksMetaOperatorDescriptor.java M hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/work/StartTasksWork.java 2 files changed, 2 insertions(+), 10 deletions(-) Approvals: Anon. E. Moose #1000171: Jenkins: Verified; No violations found; ; Verified Murtadha Hubail: Looks good to me, approved diff --git a/hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/meta/AlgebricksMetaOperatorDescriptor.java b/hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/meta/AlgebricksMetaOperatorDescriptor.java index 25a4aea..b4d23fc 100644 --- a/hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/meta/AlgebricksMetaOperatorDescriptor.java +++ b/hyracks-fullstack/algebricks/algebricks-runtime/src/main/java/org/apache/hyracks/algebricks/runtime/operators/meta/AlgebricksMetaOperatorDescriptor.java @@ -65,13 +65,7 @@ @Override public String toString() { - StringBuilder sb = new StringBuilder(); - sb.append("Asterix { \n"); - for (IPushRuntimeFactory f : pipeline.getRuntimeFactories()) { - sb.append(" " + f.toString() + ";\n"); - } - sb.append("}"); - return sb.toString(); + return "AlgebricksMeta " + Arrays.toString(pipeline.getRuntimeFactories()); } @Override diff --git a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/work/StartTasksWork.java b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/work/StartTasksWork.java index f818c0d..46b176e 100644 --- a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/work/StartTasksWork.java +++ b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/work/StartTasksWork.java @@ -139,9 +139,7 @@ ActivityId aid = tid.getActivityId(); ActivityCluster ac = acg.getActivityMap().get(aid); IActivity han = ac.getActivityMap().get(aid); - if (LOGGER.isInfoEnabled()) { - LOGGER.info("Initializing " + taId + " -> " + han); - } + LOGGER.info("Initializing {} -> {} for {}", taId, han, jobId); final int partition = tid.getPartition(); List<IConnectorDescriptor> inputs = ac.getActivityInputMap().get(aid); task = null; -- To view, visit https://asterix-gerrit.ics.uci.edu/2295 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b951e11a9716eb11832d18eba57b124faf146b1 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]>
