pjfanning commented on PR #385: URL: https://github.com/apache/incubator-pekko/pull/385#issuecomment-1588109149
@He-Pin @mdedetrich @alexandru do any of you see any reason not to merge this? I have one minor change request (about adding a tailrec annotation). Other than that, I think these changes tick the boxes that we need to fix the security issue and looks better from a thread safety respect than what is currently there. I'm sort of anxious to draw a line under this and move onto an RC. I'm expecting that we'll need at least a couple of RCs to allow the Incubator team to fully vet the licensing and other issues and it would be great to start that process this week or next week. As we draw into summer, we could find ourselves short of people to do the reviews and votes. -- 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]
