I'm looking a little bit at the source code today and there is something
that I don't understand.

In the class SocketAcceptor.Worker, in the catch block there is a
"Thread.sleep(1000);". I don't understand why you do that. What problem
does this solve? Could you add a comment in the source that tells why
this sleep is needed? That would be the best place to document this as
its purpose is not obvious. Should I create a jira issue for that?

} catch (IOException e) {
   ExceptionMonitor.getInstance().exceptionCaught(e);

   try {
       Thread.sleep(1000);
   } catch (InterruptedException e1) {
       ExceptionMonitor.getInstance().exceptionCaught(e1);
   }
}

_________________________________________________________________
Ta' på udsalg året rundt på MSN Shopping: http://shopping.msn.dk - her finder du altid de bedste priser

Reply via email to