JPA EntityManger выполняет вставку базы данных при слиянии?

Question

Я использую JPA с Hibernate внизу, и у меня возникают проблемы с слиянием для работы, но прежде чем я опишу проблемы, с которыми я сталкиваюсь с JPA, позвольте мне выложить то, что я пытаюсь достичь, на случай, если мои проблемы связаны с моим подходом Полем

У меня есть данные из системы, которые мне нужно поместить в другую систему.Для этого я читаю данные, затем создаю новые объекты ORM на основе этих данных, а затем хочу сохранить объекты ORM в базе данных.Теперь, когда база данных пуста, программа работает без проблем с помощью простого вызова em.persist(object).Однако я хочу иметь возможность запускать этот процесс, когда в базе данных есть данные, добавляя новые данные и обновляя старые данные по мере необходимости, именно здесь у меня возникают проблемы.

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

ERROR JDBCExceptionReporter - Violation of UNIQUE KEY constraint 'UK-SubStuff-StuffId-SubStuffNumber'. Cannot insert duplicate key in object 'SubStuff'.

Мне кажется странным, что вызов em.merge() пытается вставить вместо обновления (я подтвердил это с помощью журнала SQL).

Hibernate: insert into SubStuff (SubStuffNumber, StuffId, Name, TypeId) values (?, ?, ?, ?)

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

Ниже приведен мой код, извините за грубость, я попытался задокументировать здесь несколько обходных путей.

private void storeData(Collection<Stuff> Stuffs) {

    for (Stuff stuff : Stuffs) {
        //I think this first block can be safely ignored, as I am having no issues with it
        //  Left it in just in case someone more experianced then I sees the root of the issue here.
        Collection<SubStuff> subStuffs = stuff.getSubStuffCollection();
        for (SubStuff s : subStuffs) {
            //Persist SubStuff Type, which DOES NOT cascade,
            //  due to it not having an internal SubStuff collection
            Query q = em.createNamedQuery("SubStuffType.findByType");
            q.setParameter("type", f.getTypeId().getType());
            try {
                SubStuffType sst = (SubStuffType) q.getSingleResult();
                s.setTypeId(sst);
            } catch (NoResultException ex) {
                if (logger.isDebugEnabled()) logger.debug("SubStuff Type not found, persisting");
                em.persist(s.getTypeId());
            }
        }

        if (em.find(Stuff.class, stuff.getId()) == null) {
            //Persist on Stuffs will cascade to SubStuffs
            em.persist(stuff);
        } else {
            //  Failing to merge SubStuff, tries to insert duplicate
            //  Merge SubStuff first
            // The block below is my attempt to merge the SubStuff Collection before merging Stuff,
            //  it creates the same isuse as a straight merge of Stuff.
            Collection<SubStuff> mergedSubStuffs = new ArrayList<SubStuff>(SubStuffs.size());
            for (SubStuff s : SubStuffs) {
                Query q = em.createNamedQuery("SubStuff.findBySubStuffNumberStuffId");
                q.setParameter("SubStuffNumber", s.getSubStuffNumber());
                q.setParameter("StuffId", stuff.getId());
                try {
                    SubStuff subStuff = (SubStuff) q.getSingleResult();
        // -----> Merge fails, with an duplicate insert error
                    SubStuff mergedSubStuff = em.merge(s);
                    mergedSubStuffs.add(mergedSubStuff);
                } catch (NoResultException ex) {
                    throw ex;
                }
            }
            stuff.setSubStuffCollection(mergedSubStuffs);

        // -----> This will fails with same error as above, if I remove the attempt
            //  to merge the sub objects
            em.merge(stuff);
        }
    }
}

Если кто-нибудь с опытом JPA может мне помочь, я буду очень признателен.Различия между saveOrUpdate() в Hibernate и merge() в JPA, очевидно, сбивают меня с толку, но, несмотря на прочтение нескольких статей о слиянии в EnityManger, я до сих пор не могу понять, что здесь происходит.

Спасибо за ваше время.

Answer 1

Проклятие StackOverflow снова ударило.Поработав над этой проблемой около суток, я решил опубликовать этот вопрос, через 20 минут у меня случился момент эврики, и я решил его.Частично потому, что я достаточно прояснил свои мысли, чтобы опубликовать вопрос.Размышляя о том, какая информация может иметь отношение к вопросу, я понял, что виноваты мои автоматически сгенерированные ключи (или, правильно, моя глупая необработка их при слиянии).

Моя проблема в том, что SubStuff (худшая схема замены именования, извините за это) имеет автоматически сгенерированный искусственный первичный ключ.Поэтому при слиянии мне нужно было сделать;

SubStuff subStuff = (SubStuff) q.getSingleResult();
//++++++++
s.setId(subStuff.getId());
//++++++++

//The following code can be removed.
//--- SubStuff mergedSubStuff = em.merge(f);
//--- mergedSubStuffs.add(mergedSubStuff);

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

Вызов слияния можно упростить до простого вызова em.merge(stuff), так как это будет каскадировать объекты подматериалов, а коллекцию mergedsubstuff можно будет удалить целиком, поскольку мы больше не выполняем слияние внутри этого цикла.

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

Answer 2

Ваша проблема связана со следующими парой строк:

Collection<SubStuff> mergedSubStuffs = new ArrayList<SubStuff>(SubStuffs.size());
...
stuff.setSubStuffCollection(mergedSubStuffs);

JPA будет рассматривать новую коллекцию как совершенно новый набор подобъектов и всегда будет вставлять их как таковые.Продолжайте работать с исходной коллекцией SubStuff внутри сущности Stuff, и все будет в порядке.