Yes, that was wrong. I changed it to:
if (retry) {
// this usually means an NFS mount has failed;
// arrange to try again later.
//
transition_time = DELAYED;
goto leave;
}
-- David
On 25-Apr-2011 6:53 AM, Travis Desell wrote:
> Going through the validator code I came across this:
>
> 204 check_pair(result, canonical_result, retry);
> 205 if (retry) transition_time = DELAYED;
> 206 update_result = false;
> 207
> 208 if (result.outcome == RESULT_OUTCOME_VALIDATE_ERROR) {
> 209 update_result = true;
> 210 }
> 211
> 212 // this might be last result, so let transitioner
> 213 // trigger file delete etc. if needed
> 214 //
> 215 transition_time = IMMEDIATE;
>
> What's the point of setting transition_time to DELAYED on line 205 if it's
> just changed to IMMEDIATE on 215. Could this be a bug?
>
> ----------------------------------------------------------------------------------------------------------
> Travis Desell<deselt @ cs.rpi.edu> 1-518-867-1054
> Adjunct Professor& Postdoctoral Research Assistant
> Rensselaer Polytechnic Institute, 110 8th Street, Troy NY 12180, USA
> http://www.cs.rpi.edu/~deselt/
> MilkyWay@Home ( http://milkyway.cs.rpi.edu/ )
> DNA@Home ( http://dnahome.cs.rpi.edu/ )
> Worldwide Computing Laboratory ( http://wcl.cs.rpi.edu/ )
> ----------------------------------------------------------------------------------------------------------
>
> _______________________________________________
> boinc_dev mailing list
> [email protected]
> http://lists.ssl.berkeley.edu/mailman/listinfo/boinc_dev
> To unsubscribe, visit the above URL and
> (near bottom of page) enter your email address.
_______________________________________________
boinc_dev mailing list
[email protected]
http://lists.ssl.berkeley.edu/mailman/listinfo/boinc_dev
To unsubscribe, visit the above URL and
(near bottom of page) enter your email address.
