glassfish
  1. glassfish
  2. GLASSFISH-19914

BATCH RI: Abandoning a Job sets the StartTime of the Job execution to Current Time

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0_b79
    • Fix Version/s: 4.0_b83
    • Component/s: batch
    • Labels:
      None

      Description

      Tested with Glassfish 4.0 Build 80

      Steps:

      1) Abandon a Failed/Completed Job.

      Issue --> Abandon a Job sets the Start time of the Job to the current time(). Not sure this is expected. As a result of this StartTime of a Job is greater than End time(sample output of list-batch-job-executions shown below)

      $ bin/asadmin list-batch-job-executions -x 357
      JOBNAME EXECUTIONID STARTTIME ENDTIME BATCHSTATUS EXITSTATUS
      simple-batchlet-job 357 Mon Mar 18 18:41:54 IST 2013 Mon Mar 18 17:42:30 IST 2013 ABANDONED FAILED
      Command list-batch-job-executions executed successfully.

        Activity

        Hide
        ScottKurz added a comment -

        In looking at the code, it seems we are updating the start time in a couple of places where we shouldn't, including this case.

        Let me try to see if there was a reason behind this before jumping to make the change..

        Show
        ScottKurz added a comment - In looking at the code, it seems we are updating the start time in a couple of places where we shouldn't, including this case. Let me try to see if there was a reason behind this before jumping to make the change..
        Hide
        ScottKurz added a comment -

        I think this is fixed in 1.0-b22. I came at it from a different angle so didn't run this exact test.... will look for confirmation.

        Show
        ScottKurz added a comment - I think this is fixed in 1.0-b22. I came at it from a different angle so didn't run this exact test.... will look for confirmation.
        Hide
        Mahesh Kannan added a comment -

        According to Scott, this should have been fixed in b22

        Show
        Mahesh Kannan added a comment - According to Scott, this should have been fixed in b22

          People

          • Assignee:
            Mahesh Kannan
            Reporter:
            arunkumar_s
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: