Hi John,

I may be able to shed some light on this problem.

A while back I had a problem whereby if there was an exception in the job
itself the line

je.setActive(false)

was not being run. I asked this to be fixed and suggested putting the above
line in the finally clause. After thinking about the problem again I thought
that this line should have stayed where it was and instead inserted the
patch in the exception clause. I feel that I introduced a new problem while
trying to solve another :-(

/colin

<original code before my comments>
            if (! je.isActive())
            {
                je.setActive(true);
                ScheduledJobLoader.getInstance().exec(je, je.getTask());
                je.setActive(false);
            }
        }
        catch(Exception e)
        {
            org.apache.turbine.util.Log.error("Error in WorkerThread " +
                                              "for Task: " +
                                              je.getTask() + ": "+ e);
        }
</original code before my comments>

<preffered patch?>
            if (! je.isActive())
            {
                je.setActive(true);
                ScheduledJobLoader.getInstance().exec(je, je.getTask());
                 je.setActive(false);
            }
        }
        catch(Exception e)
        {
            if (je.isActive())
            {
                    je.setActive(false);
            }
              org.apache.turbine.util.Log.error("Error in WorkerThread " +
                                              "for Task: " +
                                              je.getTask() + ": "+ e);
        }
</preffered patch?>

----- Original Message -----
From: "John Thorhauer" <[EMAIL PROTECTED]>
To: "Turbine" <[EMAIL PROTECTED]>
Sent: Wednesday, February 07, 2001 4:19 PM
Subject: scheduled jobs problem


> Something has changed in the WorkerThread for jobs so that it is no longer
> waiting for the job to finish before in runs the following:
>
> je.setActive(false)
>
> So my jobs are now overlapping eachother and causing problems.  Can anyone
> help me with this one?
>
> Thanks,
> John
> --
> ********************************
> ** John Thorhauer
> ** [EMAIL PROTECTED]
> ** take a look at:
> **  http://tambora.zenplex.org
> **  http://www.zenplex.org
> **  http://www.zenplex.com
> ********************************
>
>
>
> ------------------------------------------------------------
> To subscribe:        [EMAIL PROTECTED]
> To unsubscribe:      [EMAIL PROTECTED]
> Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
> Problems?:           [EMAIL PROTECTED]
>
>



------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to