[Seaside-dev] [Dolphin] Use of Delay on CTPusher class>>startUp and pingProcess

Julian Fitzell jfitzell at gmail.com
Sat Sep 19 08:00:32 UTC 2009


On 9/18/09, Udo Schneider <Udo.Schneider at homeaddress.de> wrote:
> Sebastian,
>
> > pingProcess
> >        self allInstancesDo: [ :each | each ping ].
> >        delay wait
> >
> > I think "delay wait" can be safely replaced by "Delay forSeconds: 15"
> > (removing it from startUp).
> >
> >
> You're right ... as long as you mean " (Delay forSeconds: 15) wait" :-)
>
> BTW: Could you replace "self allInstancesDo:" with "self allInstances do:"
> ... at least in my image #allInstancesDo: is not defined.

We need to be careful with doing anything with all instances actually
because it's a terrible idea on GemStone. But I guess Comet depends on
Comanche and GS is using Swazoo so it's probably ok in this case...
still it can't be the most efficient implementation I would think.

Julian


More information about the seaside-dev mailing list