mgodave commented on issue #1355: Remove as many Thread.sleep calls from REST 
endpoint
URL: https://github.com/apache/incubator-pulsar/pull/1355#issuecomment-371242117
 
 
   Feel free to close it.
   
   On Mar 7, 2018 11:43, "Matteo Merli" <notificati...@github.com> wrote:
   
   > The point is, sleeping a thread in the REST endpoint is a resource hog and
   > generally not a great idea.
   >
   > I'm not debating that :)
   >
   > I think I see what you are saying. Either way, scheduling a return after
   > the sync timeout would be a better option.
   >
   > Definitely. It's still hacky but at least we don't keep the thread hanging.
   >
   > ?
   > You are receiving this because you were mentioned.
   > Reply to this email directly, view it on GitHub
   > 
<https://github.com/apache/incubator-pulsar/pull/1355#issuecomment-371240890>,
   > or mute the thread
   > 
<https://github.com/notifications/unsubscribe-auth/AAh9FgmREEPx8-ZsIDyQWCaI3jUr8aEnks5tcCpAgaJpZM4Sg5AV>
   > .
   >
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to