Wrong logging of JobID in MapReduceExecutable.java

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");