clesaec commented on PR #2090: URL: https://github.com/apache/avro/pull/2090#issuecomment-1431023236
Big hammer for small part of code that could be fixed by simple ReentrantReadWriteLock ... But, at least, it will be able to be used by other part of Avro. - Do we keep "springframework" in package name ? (i don't know what is the usual practice on it ? @RyanSkraba ? - as CI result say, run "'mvn spotless:apply'" - May be copy also some unit tests from springframework for this new collection class, as [this one for example](https://github.com/spring-projects/spring-framework/blob/main/spring-core/src/test/java/org/springframework/util/ConcurrentReferenceHashMapTests.java) ? -- 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]
