datamapper / do

DataObjects
147 stars 74 forks source link

avoided dynamically creating class methods. Apparently, in JRuby at least, there is a lock on that. #41

Closed patcheng closed 11 years ago

patcheng commented 11 years ago

I was testing with JDNI (Commons DBCP) with unlimited database connection. Might not be as bad if using DataMapper's connection pool.

dkubb commented 11 years ago

@dbussink what do you think about this change? It looks great to me, and it removes the redefinition of the method.