JPA가 연관관계가 많을 때 조인으로 인해 동일 Row가 중복 생성되고, 결국 엔티티가 N배수로 반복되어 조회되는 문제

[YONGHYNG]

무한 조회

• JPA가 연관관계가 많을 때 조인으로 인해 동일 Row가 중복 생성되고, 결국 엔티티가 N배수로 반복되어 조회되는 문제
• 이 문제 때문에 서비스 단에서 DTO 변환을 잘못 하면 무한 조인/중복 데이터 폭발이 생기고, 메모리도 무지하게 먹어.

DepotConverter

package kr.co.funzin.fsm.converter;

import jakarta.annotation.PostConstruct;
import kr.co.funzin.fsm.converter.mapper.ClusterSimpleMapper;
import kr.co.funzin.fsm.converter.mapper.CreatorAndModifierMapper;
import kr.co.funzin.fsm.converter.mapper.JobMapper;
import kr.co.funzin.fsm.converter.mapper.VehicleMapper;
import kr.co.funzin.fsm.entity.Depot;
import lombok.RequiredArgsConstructor;
import lombok.extern.log4j.Log4j2;
import org.modelmapper.Converters;
import org.modelmapper.ModelMapper;
import org.modelmapper.TypeMap;
import org.springframework.stereotype.Component;

import java.util.Objects;
import java.util.UUID;

@Log4j2
@RequiredArgsConstructor
@Component
public class DepotConverter {

    private final ModelMapper mapper;
    private final CreatorAndModifierMapper accountMapper;
    private final JobMapper jobMapper;
    private final VehicleMapper vehicleMapper;
    private final ClusterSimpleMapper clusterSimpleMapper;
    private TypeMap<Depot, Depot> map;


    @PostConstruct
    public void init() {
        this.map = this.mapper.createTypeMap(Depot.class, Depot.class, UUID.randomUUID().toString());
        this.map.addMappings(mapper -> {
            mapper.using(this.accountMapper).map(Depot::getCreator, Depot::setCreator);
            mapper.using(this.accountMapper).map(Depot::getModifier, Depot::setModifier);
            mapper.using(Converters.Collection.map(this.jobMapper)).map(Depot::getJobs, Depot::setJobs);
            mapper.using(Converters.Collection.map(this.vehicleMapper)).map(Depot::getVehicles, Depot::setVehicles);
            mapper.using(Converters.Collection.map(this.clusterSimpleMapper)).map(Depot::getClusters, Depot::setClusters);

        });
    }


    public Depot convert(Depot depot) {
        try {
            if (Objects.nonNull(depot)) {
                return this.map.map(depot);
            }
        } catch (Exception ignored) {
        }
        return null;
    }
}

DepotMapper

package kr.co.funzin.fsm.converter.mapper;

import jakarta.annotation.PostConstruct;
import kr.co.funzin.fsm.entity.Depot;
import lombok.RequiredArgsConstructor;
import lombok.extern.log4j.Log4j2;
import org.modelmapper.AbstractConverter;
import org.modelmapper.Converters;
import org.modelmapper.ModelMapper;
import org.modelmapper.TypeMap;
import org.springframework.stereotype.Component;

import java.util.Objects;
import java.util.UUID;

@SuppressWarnings({"java:S1117", "java:S108"})
@Log4j2
@RequiredArgsConstructor
@Component
public class DepotMapper extends AbstractConverter<Depot, Depot> implements Converters.Converter<Depot, Depot>  {

    private final ModelMapper mapper;
    private TypeMap<Depot, Depot> map;

    @PostConstruct
    public void init() {
        this.map = this.mapper.createTypeMap(Depot.class, Depot.class, UUID.randomUUID().toString());
        this.map.addMappings(mapper -> {
            mapper.skip(Depot::setCreatedBy);
            mapper.skip(Depot::setCreatedAt);
            mapper.skip(Depot::setModifiedBy);
            mapper.skip(Depot::setModifiedAt);
        });
    }

    @Override
    public Depot convert(Depot depot) {
        try {
            if (Objects.nonNull(depot)) {
                return this.map.map(depot);
            }
        } catch (Exception ignored) {
        }

        return null;
    }

}

DepotSimpleMapper

package kr.co.funzin.fsm.converter.mapper;

import jakarta.annotation.PostConstruct;
import kr.co.funzin.fsm.entity.Depot;
import lombok.RequiredArgsConstructor;
import lombok.extern.log4j.Log4j2;
import org.modelmapper.AbstractConverter;
import org.modelmapper.Converters;
import org.modelmapper.ModelMapper;
import org.modelmapper.TypeMap;
import org.springframework.stereotype.Component;

import java.util.Objects;
import java.util.UUID;

@Log4j2
@RequiredArgsConstructor
@Component
public class DepotSimpleMapper extends AbstractConverter<Depot, Depot> implements Converters.Converter<Depot, Depot> {

    private final ModelMapper mapper;
    private TypeMap<Depot, Depot> map;

    @PostConstruct
    public void init() {
        this.map = this.mapper.createTypeMap(Depot.class, Depot.class, UUID.randomUUID().toString());
        this.map.addMappings(mapper -> {
            mapper.skip(Depot::setCreatedAt);
            mapper.skip(Depot::setCreatedBy);
            mapper.skip(Depot::setModifiedAt);
            mapper.skip(Depot::setModifiedBy);
            mapper.skip(Depot::setVehicles);
            mapper.skip(Depot::setClusters);
            mapper.skip(Depot::setJobs);
            mapper.skip(Depot::setCreator);
            mapper.skip(Depot::setModifier);
        });
    }

    @Override
    public Depot convert(Depot depot) {
        try {
            if (Objects.nonNull(depot)) {
                return this.map.map(depot);
            }
        } catch (Exception ignored) {
        }

        return null;
    }
}

• 엔티티 그래프 확장을 막고, 깊은 연관관계 순환을 차단하는 구조, JPA 무한 조인 문제를 해결하는 핵심 포인트
• 이 3개의 Mapper/Converter는 결국 “연관관계 깊이에 따라 필요한 필드만 복사하고, 불필요한 연관 엔티티들은 skip 함으로써, JPA가 엔티티 그래프를 무한히 확장하면서 JOIN하는 문제를 원천 차단한다.”

📌 JPA 무한 조인/무한 로딩 문제가 발생하는 원리 (요약)

• 문제는 엔티티 연관관계가 다음처럼 깊을 때

Depot → Jobs → Route → Cluster → Depot → Jobs → ...

• 양방향 / 다단계 연관관계가 많으면 JPA는 엔티티 매핑 과정에서 계층이 계속 넓어지거나 순환되면서 쿼리가 계속 발생할 수 있음
• DTO 변환할 때도 엔티티 그래프 전체를 탐색하기 때문에 발생 가능
  • 그래서 ModelMapper + skip 조합으로 이걸 제어하는 구조를 사용

DepotConverter

• “전체 Depot 엔티티를 복제하되, 내부 참조 엔티티는 각각 다른 Mapper가 담당”
  • Join의 근원

핵심 로직

mapper.using(Converters.Collection.map(this.jobMapper)).map(Depot::getJobs, Depot::setJobs);
mapper.using(Converters.Collection.map(this.vehicleMapper)).map(Depot::getVehicles, Depot::setVehicles);
mapper.using(Converters.Collection.map(this.clusterSimpleMapper)).map(Depot::getClusters, Depot::setClusters);

• Depot → Jobs 변환은 JobMapper에 위임
• Depot → Vehicles 변환은 VehicleMapper에 위임
• Depot → Clusters 변환은 ClusterSimpleMapper에 위임

💡 계층별로 Mapper가 분리되어 있어, 한 엔티티에서 너무 많은 연관 엔티티를 한 번에 deep copy 하지 않도록 제어함, 즉, 연관관계 확산을 분리해서 통제, 무한 JOIN, 중첩 복사를 방어

DepotMapper

• “Depot 엔티티에서 Auditing 정보(createdBy/at, modifiedBy/at)만 제외한 Full Clone Mapper”

mapper.skip(Depot::setCreatedBy);
mapper.skip(Depot::setCreatedAt);
mapper.skip(Depot::setModifiedBy);
mapper.skip(Depot::setModifiedAt);

• 이는 단순히 “감사 정보 제거용 mapper”지만, 핵심은 엔티티 전체를 복사하되, 깊은 연관관계까지 순회하지 않는다. 즉, 다른 연관 엔티티는 건드리지 않는다.
• 라서 “Depot → Jobs → Depot → Jobs …” 같은 순환구조가 발생할 확률이 0됨.

DepotSimpleMapper

• “조인 폭발을 막는 데 가장 큰 역할을 하는 Mapper”

mapper.skip(Depot::setVehicles);
mapper.skip(Depot::setClusters);
mapper.skip(Depot::setJobs);
mapper.skip(Depot::setCreator);
mapper.skip(Depot::setModifier);

• Depot → Vehicles 관계 제거
• Depot → Clusters 관계 제거
• Depot → Jobs 관계 제거
• Depot → Creator/Modifier 관계 제거

DepotSimpleMapper는 “Depot 단일 객체”만 남기고, 모든 연관 테이블을 끊어버리는 Mapper

이 Mapper가 사용되는 순간:

✔ Depot 내부에서
✔ Jobs, Vehicles, Clusters 엔티티 탐색이 완전히 차단됨

➡ 즉, JPA의 깊은 fetch cascade 확장이 멈춤
➡ DTO 변환 중 엔티티 순환 접근도 불가능해짐
➡ 결과적으로 무한 JOIN, N배 로딩, Deep Graph 탐색이 모두 사라짐