ivakegg commented on PR #395: URL: https://github.com/apache/commons-vfs/pull/395#issuecomment-1553416611
> -1 failed build and: Quick note but the PR would need a detailed review from more folks: I am really skeptical here, VFS was _not_ designed for thread safety; hacking some code here and there is more likely to make the code base much harder to understand and maintain. In the past, we've just told users they have to handle thread safety in their applications. Slapping "synchronized" here and there is likely to create side effects like dead-locks in existing applications. While we've that've made some changes in the past to help with thread safety (see the mailing list, jira, and PRs), doing that consistently is delicate and I am concerned about unintended side effects. The only build failure is on mac OS 8 and the error does not appear to have anything to do with my changes. I understand the _not_ designed for thread safety issues, however if you are going to supply a class loader implementation, then the underlying protocols being used will have to be thread safe. -- 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]
