Hi Dermot,
Please see my response inline.
On 06/15/10 11:02 AM, Dermot McCluskey wrote:
Hi Karen,
A few more questions came to mind about the AbstractCheckpoint class.
Apologies if these questions are already answered in the document or
if they
have previously been raised in this thread:
- what is the unit of the return value from get_progress_estimate()?
Is it a number of seconds, milliseconds, minutes, etc?
As just discussed with Dave, get_progress_estimate would return the
number of seconds it takes to execute the checkpoint on a
standardized machine. I will define the exact configuration of
this standardized system in the final draft of my document.
- are cancel() methods expected to return anything?
- are there any guidelines on how implementations of cancel() are
meant to behave?
In considering Darren's comment on providing a comprehensive discussion
about threading, I have concluded that it is not necessary to have
a cancel method. The AbstractCheckpoint class will have a variable
called "cancel_requested", with a default value of false. If a cancel
request
is made, the engine will set this variable to true.
The execute() method in the AbstractCheckpoint() call should check the value
of this variable from time to time, and do any necessary cleanup and exit
when this variable is set to true.
- are execute() methods expected to return anything?
execute() methods are not expected to return anything.
- if an execute() method raises an exception, will it be caught by the
Engine, or passed through to the main app?
The exception will be caught by the engine. If an exception is raised,
the engine assumes the checkpoint failed, and it will do all the necessary
book keeping to not execute any checkpoints that have not been executed,
unless the continue-despite-failure flag is set.
The exception from the checkpoint will be returned to the main app.
Thanks,
--Karen
Thanks,
Dermot
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
