Comment #2 on issue 731 by julian.w...@gmail.com: Add @UnitOfWork annotation
http://code.google.com/p/google-guice/issues/detail?id=731

I haven't used the @UnitOfWork part of this patch, but I don't think the fix in JpaPersistService where it increments and decrements the count is working correctly.

I thrashed the service with soapUI using a load test on several service(all using PersistFilter) in my app simultaneously. I'm not sure how, since it's a ThreadLocal variable, but that count quickly get out of whack. As a result not all entity managers are getting closed and the connection pool eventually runs out of connections. Works fine with one thread.

Perhaps it's not handling nested @Transactions correctly. I wasted a lot of time trying get this patch working.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"google-guice-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to google-guice-dev+unsubscr...@googlegroups.com.
To post to this group, send email to google-guice-dev@googlegroups.com.
Visit this group at http://groups.google.com/group/google-guice-dev.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to