Even though the question is perfectly answered by Piotr I'd like to add some practical advise on how to use guice-persist.
I've been having issues with it which were pretty hard to debug. In my application certain threads would display outdated data and sometimes EntityManager
instances were left with old dead database connections. The root cause was to be found in the way I used the @Transactional
annotation (I only used them for methods that do updates/inserts/deletes, not for read-only methods). guice-persist stores EntityManager
instances in a ThreadLocal
as soon as you call get()
on an injected Provider<EntityManager>
instance (which I did for read-only methods). However, this ThreadLocal
is only removed if you also call UnitOfWork.end()
(which normally is done by the interceptor if @Transactional
is on the method). Not doing so will leave the EM instance in the ThreadLocal so that eventually every thread in your thread pool will have an old EM instance with stale cached entities.
So, as long as you stick to the following simple rules the usage of guice-persist is straight forward:
- Always inject
Provider<EntityManager>
instead of EntityManager
directly.
- For transaction-scoped semantics: always annotate each method with
@Transactional
(even the read-only methods). This way the JpaLocalTxnInterceptor
will intercept the calls to your annotated methods making sure not only to start and commit transactions but also to set and unset EM instances in the ThreadLocal.
- For request-scoped semantics: use the
PersistFilter
servlet filter that ships with guice-persist. It will call begin()
and end()
on the UnitOfWork
for you before and after the request is done, thereby populating and cleaning up the ThreadLocal.
Hope this helps!
与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…