[Spring, React] flush( )를 사용해서 DB 오류 해결하기

 

 

관리자 정보를 수정했더니
중복 저장 에러가 발생했다.
에러의 원인을 찾고 수정해보자.

 

 

 

 

1. 에러

Duplicate entry '2-3' for key 'admin_permission.uk_admin_permission'

처음 로직은 이랬다. [ 관리자 정보 수정 ] - [ 기존 권한 삭제 ] - [새 권한 다시 저장] 흐름으로 코드를 작성했다. 그런데 생각한 흐름대로 일처리가 되지 않은 모양이다. 어디서 문제가 발생했는지 DB부터 다시 살펴봤다.

AdminService의 update 코드 중 일부

// 기존 권한 삭제
adminPermissionRepository.deleteByAdminId(id);
// 다시 저장
savePermissions(id, request.getPermissions());​

 

DB를 3개로 분리해뒀다. 관리자의 정보를 저장하는 Admin, 권한 종류를 저장하는 Permission, 관리자와 권한을 연결해주는 중간 테이블인 AdminPermission. 그리고 admin_permission 테이블에는 유니크 제약을 걸어서 관리자와 권한의 조합이 중복될 수 없도록 설정했다.

그런데 Duplicate entry '2-3'은 admin_id = 2와 permission_id = 3의 조합이 중복되었다고 알려준 것이다. 예를 들어서 2번 관리자가 3번 권한인 ADD 권한을 갖고 있는데, 이 2-3 조합을 다시 한 번 insert하려고 했다는 이야기다. 이런 일이 발생한 이유는 아마도 기존 권한 삭제가 제대로 이뤄지지 않은게 아닐까라고 추측했다. 

 

 

 

 

---

 

 

 

 

2. flush( )

• 영속성 컨텍스트(Persistence Context)의 변경 내용을 데이터베이스에 동기화하는 작업을 말한다.
• 쉽게 말해서, 메모리상에서 변경된 데이터들을 실제 데이터베이스에 쿼리(SQL)로 날려 상태를 똑같이 맞추는 과정이다.
• JPA는 성능 최적화를 위해 쿼리를 바로 날리지 않고 쓰기 지연이란 개념을 사용한다.
• flush( )가 발생하면 쓰기 지연 때문에 밀려있던 INSERT, UPDATE 등 쿼리들이 데이터베이스로 날아간다.

 

Q. flush( )는 언제 발생하는가?

A. 위에서 설명한 것과 같이 JPA는 데이터베이스에 대한 접근을 최소화하여 성능을 높이기 위해 쓰기 지연이라는 개념을 사용한다. 개발자가 객체를 생성하거나 수정, 삭제해도 즉시 DB에 쿼리를 날리지 않는다. 대신 영속성 컨텍스트 내부의 쓰기 지연 SQL 저장소에 쿼리(INSERT, UPDATE, DELETE)를 차곡차곡 모아 둔다. 이 모아둔 쿼리를 한 번에 데이터베이스로 쏟아내는 작업이 바로 flush( )이다.

flush( )는 개발자가 명시적으로 호출할 수도 있지만, 대부분 프레임워크에 의해 자동으로 발생한다. @Transcational 메서드가 정상적으로 끝나고 데이터베이스에 커밋을 하기 직전에 자동으로 호출된다. JPQL이나 QueryDSL 등을 통해 DB에 직접 조회 쿼리를 날리기 직전에 자동으로 호출된다. 메모리에만 있고 DB에는 반영되지 않은 데이터 때문에 잘못된 결과가 조회되는 것을 막기 위해서이다. 직접 호출하는 경우는, repository.flush( ) 등을 사용하면 된다.

 

 

 

 

---

 

 

 

 

3. 에러 고치기 

• AdminService

// 기존 권한 삭제
adminPermissionRepository.deleteByAdminId(id);
// 삭제 내용을 DB에 반영
adminPermissionRepository.flush();
// 새 권한 다시 저장
savePermissions(id, request.getPermissions());

 

기존 권한을 삭제를 하고 나면 flush( )로 삭제 SQL을 강제로 먼저 실행시켜서 오류가 발생하는걸 막도록 했다.

 

 

Q. 이 코드는 꽤 괜찮은 코드일까?

A. flush( )를 사용해서 오류를 일시적으로 고치긴 했지만, 이 코드가 꽤 괜찮은건지에 대해서는 잘 모르겠다. 이 코드는 테이블 중심 코드이다. 직접 테이블을 건드린다는 말이다. admin_permission 테이블의 admin_id, permission_id 행을 직접 삭제/삽입하고 있다. 이건 SQL 테이블을 수동으로 다루고 있는 것이다.

처음에 코드를 작성했을때, [ 관리자 정보 수정 ] - [ 기존 권한 삭제 ] - [ 새 권한 다시 저장 ]이 저절로 될거라고 생각했지만 아니었다. 또  flush( )는 프레임워크에서 자동으로 호출이 되어야한다. 그럼 왜 안된건지에 대해서 생각해봐야하는데, 그 이유가 테이블을 직접적으로 건드리고 외래키 ID를 직접 넣는 것이 문제인게 아닌지 추측해볼 수 있다. 

// 외래키를 직접 넣은 코드

adminPermission.setAdminId(adminId);
adminPermission.setPermissionId(permission.getId());

JPA는 원래 테이블 행을 직접 조작하기보다 엔티티 객체의 상태와 관계를 바꾸면 Hibernate가 SQL로 변환해주는 구조이다. 그러니까 지금 코드는 JPA를 제대로 활용하지 못한다고 봐야한다. 

// 엔티티 관계를 명확하게 한 코드

adminPermission.setAdmin(admin);
adminPermission.setPermission(permission);

위의 코드처럼 바꾼다면 객체 관계가 코드상에서 들어나게 된다. 그러면 테이블 중심 코드에서 객체 관계 중심 코드로 변경될 수 있다. 물론 위의 코드로 바꾸려면 전반적인 수정이 필요할 것이다.