XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636457266
> Looks like you have some merge conflicts? Sorry for that. will fix it immediately. > Also I think this will resolve BEANUTILS-509: #21 I had no experience in using function Collections.synchronizedMap(), so I cannot tell about performance about that. Though I think this pr and that one are solving similiar things, though BEANUTILS-509 focus more on thread-safe, and this pr focus more on performance. And IMO if using ConcurrentHashMap, we can gain both thread-safe and performance. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org