From cbb7890a9eaf01dae6cf95f0a0e034c4c0ab0765 Mon Sep 17 00:00:00 2001 From: Deepak Shivamurthy <107634330+deepak-shivamurthy@users.noreply.github.com> Date: Thu, 19 Jan 2023 04:49:40 +0530 Subject: [PATCH] Fix logging to capture jobControlId ### What changes are proposed in this pull request? https://github.com/Alluxio/new-contributor-tasks/issues/643 Fix logging to capture jobControlId ### Why are the changes needed? this will help in getting jobControlId in the logs, and will help in getting status information about the job ### Does this PR introduce any user facing changes? its just log level changes pr-link: Alluxio/alluxio#16780 change-id: cid-551cf01210bb24d39239199adeb398201f67c184 --- .../main/java/alluxio/master/job/tracker/CmdJobTracker.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dora/job/server/src/main/java/alluxio/master/job/tracker/CmdJobTracker.java b/dora/job/server/src/main/java/alluxio/master/job/tracker/CmdJobTracker.java index 0a555ca66034..2b7e6246d47f 100644 --- a/dora/job/server/src/main/java/alluxio/master/job/tracker/CmdJobTracker.java +++ b/dora/job/server/src/main/java/alluxio/master/job/tracker/CmdJobTracker.java @@ -114,7 +114,8 @@ private void runDistributedCommand(CmdConfig cmdConfig, long jobControlId) MigrateCliConfig migrateCliConfig = (MigrateCliConfig) cmdConfig; AlluxioURI srcPath = new AlluxioURI(migrateCliConfig.getSource()); AlluxioURI dstPath = new AlluxioURI(migrateCliConfig.getDestination()); - LOG.info("run a dist cp command, cmd config is " + cmdConfig); + LOG.info("run a dist cp command, job control id: {}, cmd config: {}", + jobControlId, cmdConfig); cmdInfo = mMigrateCliRunner.runDistCp(srcPath, dstPath, migrateCliConfig.getOverWrite(), migrateCliConfig.getBatchSize(), jobControlId);