garydgregory edited a comment on pull request #101: URL: https://github.com/apache/commons-dbcp/pull/101#issuecomment-851641187
> > Hi @arturobernalg > > Why? You need a clear explanation of why `Collections.synchronizedList` is better than `Vector` or `CopyOnWriteArrayList`. > > Can changing this be likely to have ramification beyond what the current tests assert? > > I plan on cutting a release candidate very soon so this one might have to wait for the next go around. > > Hi @garydgregory > We won using the List interface and all the implementations. We don't lose the synchronized behavior because it create by Collections.synchronizedList. > And seems logical --> https://stackoverflow.com/a/1386288/4495592 > Maybe I'm wrong. > TY Yes, i understand the separation of concerns; but I am wondering if it worth looking into synchronization at a higher level, maybe, maybe not. In the meantime, I'll take another look at this in more detail later today. -- 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