swamirishi commented on PR #5552: URL: https://github.com/apache/ozone/pull/5552#issuecomment-1799953552
> Thanks @swamirishi for the patch, @aryangupta1998 for the review. > > > Can this PR be merged? > > Yes. > > When merging PRs, I usually wait until CI for the previous commit has (mostly) finished, to avoid completely filling our queue (i.e. to leave some capacity for other PRs). > > I also try to avoid merging PRs less than 24 hours after they were opened, to give contributors from all time zones a chance to take a look at them. (With the exception of fixes for CI breakage.) Sure thanks a lot for the pointer. Will keep this in mind for other patches. -- 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]
