ArafatKhan2198 commented on PR #5707: URL: https://github.com/apache/ozone/pull/5707#issuecomment-1842720584
Thanks for your effort on this, @adoroszlai ! The patch looks to be in good shape from my perspective. I do have one minor comment. It would be good if we could include a small JavaDoc comment for the TestBlockOutputStream class. This comment should mention purpose of test class and that `flushDelay` is parametrised. The other individual methods have good explanations in the comments, so I believe there's no need for individual JavaDocs. -- 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]
