Uploaded image for project: 'Kylin'
  1. Kylin
  2. KYLIN-1639

Wrong logging of JobID in MapReduceExecutable.java

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: all, v1.5.1
    • Fix Version/s: v1.3.1, v1.5.2
    • Component/s: Job Engine
    • Labels:
      None

      Description

      Hello,

      during debugging of an issue where MR Build Job (Step 2) errors - even the MR job is SUCCESS - I found out strange entry in the Kylin log:

      2016-04-30 01:40:31,358 INFO [pool-2-thread-1] execution.AbstractExecutable:107 : mr_job_id:null

      After looking at the source code of Kylin 1.5.1 I found that there is probably an code-error in MapReduceExecutable.java on line 107:

      logger.info("mr_job_id:" + extra.get(ExecutableConstants.MR_JOB_ID + " resumed"));

      I believe the correct logging logic is suposed to be:

      logger.info("mr_job_id:" + extra.get(ExecutableConstants.MR_JOB_ID) + " resumed");

        Attachments

          Activity

            People

            • Assignee:
              lidong_sjtu Dong Li
              Reporter:
              calaba@gmail.com Richard Calaba
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: