2012-01-06 2 views
9

Использование JPA с реализацией EclipseLink.JPA В настоящий момент транзакция не активна

Код:

try{ 
    if(!em.getTransaction().isActive()) 
     em.getTransaction().begin(); 
    System.out.println(2); 
    em.persist(currentUser); 
    System.out.println(3); 
    if (em.getTransaction().isActive()){ 
     System.out.println("IS ACTIVE"); 
    } else { 
     System.out.println("NO ACTIVE"); 
    } 
    em.getTransaction().commit(); 
    System.out.println(4); 
    } catch (Exception e){ 
    completed = false; 
    em.getTransaction().rollback(); 
    System.out.println("ERROR: " + e.getMessage()); 
} 

Ошибка:

INFO: persistOne - Enter 
INFO: 2 
INFO: 3 
INFO: IS ACTIVE 
INFO: [EL Warning]: 2012-01-06 14:45:59.221--UnitOfWork(12492659)--java.lang.IllegalStateException: During synchronization a new object was found through a relationship that was not marked cascade PERSIST: [email protected] 

WARNING: #{accountController.performRegister}: java.lang.IllegalStateException: 
Exception Description: No transaction is currently active 
javax.faces.FacesException: #{accountController.performRegister}: java.lang.IllegalStateException: 
Exception Description: No transaction is currently active 
    at com.sun.faces.application.ActionListenerImpl.processAction(ActionListenerImpl.java:118) 
    at javax.faces.component.UICommand.broadcast(UICommand.java:315) 
    at javax.faces.component.UIViewRoot.broadcastEvents(UIViewRoot.java:794) 
    at javax.faces.component.UIViewRoot.processApplication(UIViewRoot.java:1259) 
    at com.sun.faces.lifecycle.InvokeApplicationPhase.execute(InvokeApplicationPhase.java:81) 
    at com.sun.faces.lifecycle.Phase.doPhase(Phase.java:101) 
    at com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:118) 
    at javax.faces.webapp.FacesServlet.service(FacesServlet.java:593) 
    at org.apache.catalina.core.StandardWrapper.service(StandardWrapper.java:1539) 
    at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:281) 
    at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175) 
    at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:655) 
    at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:595) 
    at com.sun.enterprise.web.WebPipeline.invoke(WebPipeline.java:98) 
    at com.sun.enterprise.web.PESessionLockingStandardPipeline.invoke(PESessionLockingStandardPipeline.java:91) 
    at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:162) 
    at org.apache.catalina.connector.CoyoteAdapter.doService(CoyoteAdapter.java:330) 
    at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:231) 
    at com.sun.enterprise.v3.services.impl.ContainerMapper.service(ContainerMapper.java:174) 
    at com.sun.grizzly.http.ProcessorTask.invokeAdapter(ProcessorTask.java:828) 
    at com.sun.grizzly.http.ProcessorTask.doProcess(ProcessorTask.java:725) 
    at com.sun.grizzly.http.ProcessorTask.process(ProcessorTask.java:1019) 
    at com.sun.grizzly.http.DefaultProtocolFilter.execute(DefaultProtocolFilter.java:225) 
    at com.sun.grizzly.DefaultProtocolChain.executeProtocolFilter(DefaultProtocolChain.java:137) 
    at com.sun.grizzly.DefaultProtocolChain.execute(DefaultProtocolChain.java:104) 
    at com.sun.grizzly.DefaultProtocolChain.execute(DefaultProtocolChain.java:90) 
    at com.sun.grizzly.http.HttpProtocolChain.execute(HttpProtocolChain.java:79) 
    at com.sun.grizzly.ProtocolChainContextTask.doCall(ProtocolChainContextTask.java:54) 
    at com.sun.grizzly.SelectionKeyContextTask.call(SelectionKeyContextTask.java:59) 
    at com.sun.grizzly.ContextTask.run(ContextTask.java:71) 
    at com.sun.grizzly.util.AbstractThreadPool$Worker.doWork(AbstractThreadPool.java:532) 
    at com.sun.grizzly.util.AbstractThreadPool$Worker.run(AbstractThreadPool.java:513) 
    at java.lang.Thread.run(Thread.java:662) 
Caused by: javax.faces.el.EvaluationException: java.lang.IllegalStateException: 
Exception Description: No transaction is currently active 
    at javax.faces.component.MethodBindingMethodExpressionAdapter.invoke(MethodBindingMethodExpressionAdapter.java:102) 
    at com.sun.faces.application.ActionListenerImpl.processAction(ActionListenerImpl.java:102) 
    ... 32 more 
Caused by: java.lang.IllegalStateException: 
Exception Description: No transaction is currently active 
    at org.eclipse.persistence.internal.jpa.transaction.EntityTransactionImpl.rollback(EntityTransactionImpl.java:122) 
    at com.maze.service.UserService.persistOne(UserService.java:63) 
    at com.maze.controller.request.AccountController.performRegister(AccountController.java:62) 
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) 
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) 
    at java.lang.reflect.Method.invoke(Method.java:597) 
    at com.sun.el.parser.AstValue.invoke(AstValue.java:234) 
    at com.sun.el.MethodExpressionImpl.invoke(MethodExpressionImpl.java:297) 
    at com.sun.faces.facelets.el.TagMethodExpression.invoke(TagMethodExpression.java:105) 
    at javax.faces.component.MethodBindingMethodExpressionAdapter.invoke(MethodBindingMethodExpressionAdapter.java:88) 
    ... 33 more 

Странная вещь, что сделка является активным, но следующее, что происходит ошибка бездействия.

EDIT EntityManager Singleton: общественного класса EntityManagerSingleton {

private EntityManagerSingleton(){ 
} 

private static class EMSingletonHolder{ 
    private static final EntityManagerFactory emf = Persistence.createEntityManagerFactory("Maze"); 
    private static final EntityManager em = emf.createEntityManager(); 
} 

public static EntityManager getInstance(){ 
    return EMSingletonHolder.em; 
} 

* Получение эм Instance: *

public abstract class AbstractService { 
protected EntityManager em; 

public AbstractService(){ 
    em = EntityManagerSingleton.getInstance(); 
} 
} 

Все другие услуги увеличивается срок AbstractService

+0

Я бы обратил внимание на это - 'Во время синхронизации новый объект был обнаружен через связь, которая не была отмечена каскадом PERSIST'. – Perception

ответ

5

Судя по вашему журналу, вы используете GlassFish. Я предполагаю, что вы также используете JTA. Может быть, вы также можете указать код, который получает экземпляр EntityManager? Кроме того, отладка с помощью инструкций System.out - это что-то из прошлого, используйте отладчик, иначе ваш код трудно читать. Кроме того, ваш отступ неправильный.

Прежде всего, если вы используете JTA, взгляните на этот вопрос: Цитата

EJBException when calling entityManager.getTransaction()

из принятого ответа:

It is illegal to obtain a reference to the EntityTransaction instance associated with the EntityManager in a Java EE managed context. From the Java EE API documentation of EntityManager.getTransaction():

Return the resource-level EntityTransaction object. The EntityTransaction instance may be used serially to begin and commit multiple transactions.

Returns: 
    EntityTransaction instance Throws: 
    IllegalStateException - if invoked on a JTA entity manager 

Вот почему, если вы ожидаете какой-либо помощи, важно предоставить информацию о коде/инъекции, через которую вы получаете менеджера сущности.

Кроме того, для удобства чтения кода вы не должны иметь несколько вызовов em.getTransaction(); Вы должны назначить свой первый вызов на объект транзакции и использовать его, вроде как это:

//GOOD cause readable 
EntityTransaction tx = em.getTransaction(); 
tx.begin(); 
em.persist(someObject); 
tx.commit(); 

и не

//BAD cause hard to read 
em.getTransaction().begin(); 
em.persist(someObject); 
em.getTransaction().commit(); //NO! 
+0

Я использую resource_local, а не JTA – TGM

+0

Последняя точка вашего сообщения кажется неправильной в контексте ресурсов-локальных менеджеров сущностей. Нет причин, почему бы не вызвать getTransaction несколько раз, чтобы получить текущую транзакцию. EntityManager является однопоточным, и спецификация явно показывает пример, который делает это таким образом. – chromanoid

+0

@chromanoid Я имел в виду условия чтения кода. Подумайте, вам нужно сохранить еще 4 объекта. Вы бы назвали 'em.getTransaction(). Persist (someObject)' еще 4 раза? Или вы предпочитаете «tx.persist (someObject)»? В первом подходе каждая из строк будет читать как выполнение двух действий: получить транзакцию, сохранить объект. Хотя на самом деле то, что вы хотите сказать, это «сохранить объект». –

1

Если посмотреть на последний из этих следов стека, я вижу

at org.eclipse.persistence.internal.jpa.transaction.EntityTransactionImpl.rollback(EntityTransactionImpl.java:122) 

Это ошибка во время откат, а не во время фиксации. Мне интересно, связано ли это с предупреждением об непереносимом объекте; что-то пойдет не так в конце фиксации, так что ваша инфраструктура пытается откат в тот момент, когда это слишком поздно. Готов поспорить, если вы правильно зарегистрировали этот объект UserDetail, проблема исчезнет.

8

Обратите внимание, что последнее исключение выбрано rollback() в блоке catch.

Итак, commit() бросает следующее исключение:

java.lang.IllegalStateException: During synchronization a new object was found through a relationship that was not marked cascade PERSIST: [email protected]

Это исключение вызывает неявный откат, так что руководство rollback() в блоке улова бросает еще одно исключения из-за неактивную сделку.

Для надежной ручной очистки вам необходимо проверить, активна ли транзакция до вызова rollback() в блоке catch.

3

Ну, ваши журналы объясняют, во время фиксации Eclipselink получил исключение и откат вашей транзакции. Вот почему ваша трансакция больше не активна.

INFO: [EL Warning]: 2012-01-06 14:45:59.221--UnitOfWork(12492659)--java.lang.IllegalStateException: During synchronization a new object was found through a relationship that was not marked cascade PERSIST: [email protected]

Смежные вопросы