smengcl commented on PR #6156: URL: https://github.com/apache/ozone/pull/6156#issuecomment-1925484226
Thanks for reviewing and merging this @adoroszlai . Good catch I haven't removed the comment above as well. I think a more efficient way to achieve the same test goal without having to use `sleep()` here is to check the endpoint after the datanode has booted up **and in a stable operational state**. The former is already achieved by `restartHddsDatanode()` since it is blocking. The for-loop won't even be necessary in this case. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
