Looks ok to me.

--Sean

On 3/12/18 2:42 AM, Weijun Wang wrote:
Please take a review at

    http://cr.openjdk.java.net/~weijun/8186228/webrev.00/

Even a timeout of 30 seconds could happen, maybe because the UDP packet is 
lost. The change covers all possible output where each request has 3 chances of 
retry:

   without pre-auth   with pre-auth
   ----------------   ---------------
   OK                 OK
   OK                 ERR OK
   OK                 ERR ERR OK
   OK                 ERR ERR ERR        (failure)

   ERR OK             OK
   ....
   ERR ERR OK         ERR ERR OK
   ERR ERR OK         ERR ERR ERR        (failure)

   ERR ERR ERR                           (failure)

Thanks
Max


Reply via email to