Chukwa
  1. Chukwa
  2. CHUKWA-47

Currently the jobComplete's method on the job instrumentation class is called only for Job that succeed. need to have similar functionnality for failed Job

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      1. CHUKWA-47.patch
        3 kB
        Jerome Boulon

        Issue Links

          Activity

          Hide
          Ari Rabkin added a comment -

          Is this a Chukwa problem or a Hadoop problem?

          Show
          Ari Rabkin added a comment - Is this a Chukwa problem or a Hadoop problem?
          Hide
          Eric Yang added a comment -

          Both, the Job Tracker Instrumentation API didn't have a method for finalizeJob. The previous implementation assume completeJob was called for both succeed and failed jobs. It wasn't the case, hence the API should add a new method for finalizedJob. This change would ensure the chukwa job history log file streaming can be stopped for failed jobs.

          Show
          Eric Yang added a comment - Both, the Job Tracker Instrumentation API didn't have a method for finalizeJob. The previous implementation assume completeJob was called for both succeed and failed jobs. It wasn't the case, hence the API should add a new method for finalizedJob. This change would ensure the chukwa job history log file streaming can be stopped for failed jobs.
          Hide
          Jerome Boulon added a comment -

          Update JobInstrumentation to use the new finalizeJob method and also take advantage of the FileAdaptor.
          WRN: Current FileAdaptor should not be used before CHUKWA-44/CHUKWA-70 have been fixed

          Show
          Jerome Boulon added a comment - Update JobInstrumentation to use the new finalizeJob method and also take advantage of the FileAdaptor. WRN: Current FileAdaptor should not be used before CHUKWA-44 / CHUKWA-70 have been fixed
          Hide
          Eric Yang added a comment -

          +1 Looks good.

          Show
          Eric Yang added a comment - +1 Looks good.
          Hide
          Eric Yang added a comment -

          I just committed this, thanks Jerome.

          Show
          Eric Yang added a comment - I just committed this, thanks Jerome.

            People

            • Assignee:
              Jerome Boulon
              Reporter:
              Jerome Boulon
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development