Почему Hibernate разрешает удалять сущность, на которую ссылается FK?

Question

[Eugenue Cesarevich]

В приложении я использую Spring Data и Hibernate. База данных HSQLDB.

Есть таблицы students и сourses, связанные отношением ManyToMany. Эти отношения хранятся в таблице students_courses:

CREATE TABLE students
(
    id    INTEGER GENERATED BY DEFAULT AS SEQUENCE global_seq PRIMARY KEY,
    name  VARCHAR(255) NOT NULL,
);

CREATE TABLE courses
(
    id    INTEGER GENERATED BY DEFAULT AS SEQUENCE global_seq PRIMARY KEY,
    name  VARCHAR(255) NOT NULL,
);

CREATE TABLE students_courses
(
    student_id INTEGER NOT NULL,
    course_id   INTEGER NOT NULL,
    FOREIGN KEY (student_id) REFERENCES students (id),
    FOREIGN KEY (course_id) REFERENCES courses (id) ON DELETE CASCADE
);

Я хочу, чтобы нельзя было удалить студента, который уже участвует в каких-то курсах. Именно поэтому вот здесь: FOREIGN KEY (student_id) REFERENCES students (id) отсутствует ON DELETE CASCADE.

Я пытаюсь удалить студента с курсами из консоли HSQLDB и получаю ожидаемую ошибку: integrity constraint violation: foreign key no action; SYS_FK_10323 table: students_courses. То есть всё работает нормально. Но когда я пытаюсь удалить студента при помощи вызова delete() вот в таком репозитории:

@Repository
@Transactional(readOnly = true)
public interface StudentRepository extends JpaRepository<Student, Integer> {
    @Transactional
    @Modifying
    @Query("DELETE FROM Student s WHERE s.id=:id")
    int delete(@Param("id") int id);
}

то этот код срабатывает и не бросает ошибку! Причём я проверял - студент реально удаляется, поведение такое, как будто установлен ON DELETE CASCADE, хотя это не так. Как мне заставить Hibernate выбрасывать ошибку и не удалять сущность, на которую ссылается FK?

Comments

[Eugenue Cesarevich]

Лентюй, в консоли вот что:

Hibernate: 
    /* delete FKs in 
    join
        table */ delete 
        from
            students_courses 
        where
            (
                student_id
            ) in (
                select
                    id 
                from
                    students 
                where
                    id=?
            )
Hibernate: 
    /* DELETE 
    FROM
        Student s 
    WHERE
        s.id=:id */ delete 
        from
            students 
        where
            id=?
Hibernate: 
    /* dynamic native SQL query */ SELECT
        * 
    FROM
        students_courses

[Eugenue Cesarevich]

Лентюй, ну ему-то может и пофиг, но разве бд не должна слать его при попытки выполнения нелегитимного запроса? Когда я напрямую в бд этот запрос делаю, меня она посылает. А Hibernate почему-то нет.

[Eugenue Cesarevich]

Лентюй, что вы имеете в виду? Запрос гибернейта и мой запрос идентичны. О какой консоли вы говорите? О консоли, куда процесс работы программы выводится? Или о консоли, в которую команды вводить? Всегда так было, что если база данных не даёт что-то сделать, то гибернейт выбрасывает ошибку. Например, ограничение уникальности поля нарушается - гибернейт бросает ошибку, хотя правило уникальности указано только в бд.

Answer 1

[Eugenue Cesarevich]

Я нашёл ответ на этот вопрос на стеке.
Суть в том, что Hibernate при удалении сущности со связью ManyToMany всегда сначала удаляет её связи, а потом уже её саму. Именно поэтому никакой ошибки и не возникает.
Ссылка

Answer 2

[Орхан Гасанлы]

Добрый день!
Наверное, помимо ограничений на уровне БД можно использовать проверку средствами Java. Т.е. создать метод, который проверяет наличие курсов у юзера и выбрасывает соответствующее исключение. Соответственно, исключение будет выброшено до вызова метода delete() и будет выброшено то исключение, которе вам нужно.
Я конечно не отвечу вам наверняка, но думаю этот ответ будет для вас полезен
https://stackoverflow.com/questions/14875793/jpa-h...

JPA does offer possibility to cascade operations (merge, persist, refresh, remove) to associated entities. Logic is in JPA and does not utilize database cascades.
There is no JPA standard compliant way to do cascades with database cascades.

Т.е. скорее всего вы каскады не указали при создании сущности ManyToMany. Проверьте ваш код

Дополнено:

There is no clean cut means to do this in JPA. The following will get you what you want... You can use CascadeType.DELETE, however this annotation only applies to the objects in the EntityManager, not the database. You want to be sure that ON DELETE CASCADE is added to the database constraint. To verify, you can configure JPA to generate a ddl file. Take a look at the ddl file, you'll notice that ON DELETE CASCADE is not part of the constraint. Add ON DELETE CASCADE to actual SQL in the ddl file, then update your database schema from the ddl. This will fix your problem .

Comments

[Eugenue Cesarevich]

Да, я как раз сделал проверку в методе! Но ревьюер сказал, что можно обойтись без неё, если делегировать проверку на базу данных, т.е. сделать запись не удаляющуюся по каскаду. Т.е. я намеренно не указывал каскад в базе данных и в аннотациях. И когда я напрямую обращаюсь к бд, то получаю ошибку при удалении связанной сущности, как и планировал. А вот через репозиторий никакой ошибки не вылезает и сущность спокойно удаляется...

[Орхан Гасанлы]

Eugenue Cesarevich,

Т.е. я намеренно не указывал каскад в базе данных и в аннотациях.

А где вы тогда указываете ON DELETE CASCADE ?
И почему бы не указываете CascadeType в ManyToMany? Что насчет ссылочки, которую я скинул?

[Eugenue Cesarevich]

Орхан Гасанлы, нигде не указывал. В том и суть. Я специально его нигде не указывал, чтобы получать ошибку при попытке удаления. А Hibernate спокойно удаляет, но никакую ошибку не бросает.