Spring Data JPA с ManyToOne операция сохранения бросает org.hibernate.PersistentObjectException

Question

Я экспериментирую с Spring Data JPA, но имею проблемы с сохранением отношений ManyToOne.

@Entity 
public class Customer { 

@Id 
@GeneratedValue(strategy=GenerationType.AUTO) 
private Long id; 
private String firstName; 
private String lastName; 


@ManyToOne(cascade=CascadeType.ALL) 
private Address homeAddress; 

protected Customer() {} 
// getters and setters ... 
} 

Многие клиенты могут иметь один и тот же адрес:

@Entity 
public class Address { 

    @Id 
    @GeneratedValue(strategy = GenerationType.AUTO) 
    private Long id; 

    private String postalCode; 
    private String city; 
    private String line1; 
    private String line2; 
    private String country; 

    public Address() { 
    // TODO Auto-generated constructor stub 
    } 

    // getters and setter 
} 

CustomerRepository

import org.springframework.data.jpa.repository.JpaRepository; 

public interface CustomerRepository extends JpaRepository<Customer, Long>  { 

List<Customer> findByLastName(String lastName); 
List<Customer> findByHomeAddress(Address address); 

} 

Применение

@SpringBootApplication 
public class Application { 

private static final Logger log = LoggerFactory.getLogger(Application.class); 

public static void main(String[] args) { 
    SpringApplication.run(Application.class); 
} 

@Bean 
public CommandLineRunner demo(CustomerRepository repository) { 
    return (args) -> { 



     Address address1 = new Address("07093", "Brooklyn", "129 67th st", null, "USA"); 
     Address address2 = new Address("03333", "Qeeens", "333 67th st", null, "USA"); 
     // save a couple of customers 
     Customer jack = new Customer("Jack", "Bauer"); 

     jack.setHomeAddress(address1); 
     repository.save(jack);     
     repository.save(new Customer("Chloe", "O'Brian",address1)); 

} 
} 

Так что я пытался спасти двух клиентов с одинаковыми адресами I получить следующее исключение:

Caused by: org.springframework.dao.InvalidDataAccessApiUsageException: detached entity passed to persist: hello.Address; nested exception is org.hibernate.PersistentObjectException: detached entity passed to persist: hello.Address 
at org.springframework.orm.jpa.vendor.HibernateJpaDialect.convertHibernateAccessException(HibernateJpaDialect.java:299) ~[spring-orm-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.orm.jpa.vendor.HibernateJpaDialect.translateExceptionIfPossible(HibernateJpaDialect.java:244) ~[spring-orm-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.orm.jpa.AbstractEntityManagerFactoryBean.translateExceptionIfPossible(AbstractEntityManagerFactoryBean.java:491) ~[spring-orm-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.dao.support.ChainedPersistenceExceptionTranslator.translateExceptionIfPossible(ChainedPersistenceExceptionTranslator.java:59) ~[spring-tx-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.dao.support.DataAccessUtils.translateIfNecessary(DataAccessUtils.java:213) ~[spring-tx-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.dao.support.PersistenceExceptionTranslationInterceptor.invoke(PersistenceExceptionTranslationInterceptor.java:147) ~[spring-tx-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:179) ~[spring-aop-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.data.jpa.repository.support.CrudMethodMetadataPostProcessor$CrudMethodMetadataPopulatingMethodInterceptor.invoke(CrudMethodMetadataPostProcessor.java:133) ~[spring-data-jpa-1.10.5.RELEASE.jar:na] 
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:179) ~[spring-aop-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:92) ~[spring-aop-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:179) ~[spring-aop-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:213) ~[spring-aop-4.3.4.RELEASE.jar:4.3.4.RELEASE] 
at com.sun.proxy.$Proxy60.save(Unknown Source) ~[na:na] 
at hello.Application.lambda$0(Application.java:35) [classes/:na] 
at org.springframework.boot.SpringApplication.callRunner(SpringApplication.java:800) [spring-boot-1.4.2.RELEASE.jar:1.4.2.RELEASE] 
... 6 common frames omitted 

Как решить эту проблему? вы видите полный код на this git location

Answer 1

Добавить другую сторону отношений, сделать ее двунаправленной. В Address классе

@OneToMany(mappedBy = "homeAddress", fetch = FetchType.LAZY) 
private Set<Customer> customers 

EDIT: Вышеуказанное предложение не решить эту проблему. Ваша проблема вызвала у меня любопытство, и я проверил его в своем местном. Когда вы выполняете эти операции каждый в своей транзакции, менеджер объектов отделяет сохраненный address1 после первого сохранения (диспетчер/сеанс сущности закрывается после каждого взаимодействия с репозиторией).

Ваш метод будет работать, если, например, вы будете обертывать все операции внутри одной транзакции, которая держит диспетчер сущности открытой (что обычно делается при обслуживании, в зависимости от вашей логики отката) Я протестировал с помощью приведенного ниже кода, и он работает.

package hello; 

    import javax.transaction.Transactional; 

    import org.springframework.beans.factory.annotation.Autowired; 
    import org.springframework.stereotype.Service; 

    @Service 
    public class CustomerService { 
     @Autowired 
     private CustomerRepository customerRepository; 

     @Transactional 
     public void store(){ 
      Address address1 = new Address("07093", "Brooklyn", "129 67th st", null, "USA"); 
      Address address2 = new Address("03333", "Qeeens", "333 67th st", null, "USA"); 
      // save a couple of customers 
      Customer jack = new Customer("Jack", "Bauer"); 

      jack.setHomeAddress(address1); 

      customerRepository.save(jack);     
      customerRepository.save(new Customer("Chloe", "O'Brian", address1)); 
     } 
    } 

Кроме того, я должен был удалить Нарьяна стартер от pom.xml

Answer 2

Упорство ваши адреса первого использования addressRepository, а затем обратиться к нему.

Попробуйте

Address address1 = new Address("07093", "Brooklyn", "129 67th st", null, "USA"); 
Address address2 = new Address("03333", "Qeeens", "333 67th st", null, "USA"); 
address1 = addressRepository.save(address1); 
address2 = addressRepository.save(address2); 

// save a couple of customers 
Customer jack = new Customer("Jack", "Bauer"); 
jack.setHomeAddress(address1); 
repository.save(jack);     
repository.save(new Customer("Chloe", "O'Brian",address1)); 

PS: Я не проверял это, но это должно работать.

Answer 3

Так что это д изложение того, что на самом деле работал на меня:

1- используя CustomerRepository и AddressRepository как предложено @Abdullah Васи. это один работал до тех пор, у меня есть каскадный = CascadeType.MERGE как каскадный типа:

@ManyToOne(cascade=CascadeType.MERGE) 
private Address homeAddress; 

2- с использованием CustomerRepository и выполнить операцию из службы с @Transactional согласно @isah ответа. Опция также работала так долго, как у меня есть cascade = CascadeType.ALL в качестве каскадного типа:

@ManyToOne(cascade=CascadeType.MERGE) 
private Address homeAddress; 

службы

package hello; 

import javax.transaction.Transactional; 

import org.springframework.beans.factory.annotation.Autowired; 
import org.springframework.stereotype.Service; 

@Service 
public class CustomerService { 
    @Autowired 
    private CustomerRepository customerRepository; 

    @Transactional 
    public void store(){ 
     Address address1 = new Address("07093", "Brooklyn", "129 67th st", null, "USA"); 
     Address address2 = new Address("03333", "Qeeens", "333 67th st", null, "USA"); 
     // save a couple of customers 
     Customer jack = new Customer("Jack", "Bauer"); 

     jack.setHomeAddress(address1); 

     customerRepository.save(jack);     
     customerRepository.save(new Customer("Chloe", "O'Brian", address1)); 
    } 
} 

я считаю @isah как более оптимальное решение, так как моя работа будет в конечном итоге в классе обслуживания.