rmannibucau commented on pull request #107: URL: https://github.com/apache/maven-scm/pull/107#issuecomment-971782876
@Lonzak I'd like 1 and 2 to merge as the default when nothing is specified in our plugin tools stack, this way we are always in a safe runtime zone. The runtime warning would move from the runtime to the build time (mojo dev) since it is the only location something can be done - at runtime users will suffer from it but can't do anything without patching the plugin so warning is not really helping until the runtime execution is kept concurrent which is not safe. So I think we should lean toward: 1. threadSafe=false or unspecified = considered as false at runtime, warning at build time 2. threadSafe=true = let it be concurrent -- 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]
