Skip to content

Commit

Permalink
TEZ-4451: ThreadLevel IO Stats Support for TEZ. (#331) (Ayush Saxena …
Browse files Browse the repository at this point in the history
…reviewed by Laszlo Bodor, Steve Loughran)
  • Loading branch information
ayushtkn authored Feb 6, 2024
1 parent 5e1cdee commit 8b40858
Showing 1 changed file with 9 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@
import java.security.PrivilegedExceptionAction;
import java.util.concurrent.atomic.AtomicBoolean;

import org.apache.commons.lang.StringUtils;
import org.apache.hadoop.fs.statistics.IOStatisticsContext;
import org.apache.hadoop.fs.statistics.IOStatisticsLogging;
import org.apache.hadoop.security.UserGroupInformation;
import org.apache.tez.common.CallableWithNdc;
import org.apache.tez.common.TezCommonUtils;
Expand Down Expand Up @@ -75,6 +78,7 @@ public TaskRunner2CallableResult run() throws Exception {
LOG.info("Initializing task" + ", taskAttemptId={}", task.getTaskAttemptID());
TezUtilsInternal.setHadoopCallerContext(task.getHadoopShim(), task.getTaskAttemptID());
TezCommonUtils.logCredentials(LOG, ugi.getCredentials(), "taskInit");
IOStatisticsContext.getCurrentIOStatisticsContext().reset();
task.initialize();

if (!stopRequested.get() && !Thread.currentThread().isInterrupted()) {
Expand Down Expand Up @@ -116,6 +120,11 @@ public TaskRunner2CallableResult run() throws Exception {
// For a successful task, however, this should be almost no delay since close has already happened.
maybeFixInterruptStatus();
LOG.info("Cleaning up task {}, stopRequested={}", task.getTaskAttemptID(), stopRequested.get());
String ioStats = IOStatisticsLogging.ioStatisticsToPrettyString(
IOStatisticsContext.getCurrentIOStatisticsContext().getIOStatistics());
if (StringUtils.isNotEmpty(ioStats)) {
LOG.info("TaskAttemptId={}, {}", task.getTaskAttemptID(), ioStats);
}
task.getOutputContexts().forEach(outputContext
-> outputContext.trapEvents(new TezTrapEventHandler(outputContext,
this.tezUmbilical)));
Expand Down

0 comments on commit 8b40858

Please sign in to comment.