LogTrace
인터페이스를 먼저 만들고, 구현해보자.LogTrace
package com.example.springadvanced.trace.logtrace;
public interface LogTrace {
TraceStatus begin(String message);
void end(TraceStatus status);
void exception(TraceStatus status, Exception e);
}
FieldLogTrace(구현체)
package com.example.springadvanced.trace.logtrace;
import com.example.springadvanced.trace.TraceId;
import com.example.springadvanced.trace.TraceStatus;
import lombok.extern.slf4j.Slf4j;
@Slf4j
public class FieldLogTrace implements LogTrace {
private static final String START_PREFIX = "-->";
private static final String COMPLETE_PREFIX = "<--";
private static final String EX_PREFIX = "<X-";
private TraceId traceIdHolder; //traceId 동기화, 동시성 이슈 발생
@Override
public TraceStatus begin(String message) {
syncTraceId();
TraceId traceId = traceIdHolder;
Long startTimeMs = System.currentTimeMillis();
log.info("[{}] {}{}", traceId.getId(), addSpace(START_PREFIX, traceId.getLevel()), message);
return new TraceStatus(traceId, startTimeMs, message);
}
@Override
public void end(TraceStatus status) {
complete(status, null);
}
@Override
public void exception(TraceStatus status, Exception e) {
complete(status, e);
}
private void complete(TraceStatus status, Exception e) {
Long stopTimeMs = System.currentTimeMillis();
long resultTimeMs = stopTimeMs - status.getStartTimeMs();
TraceId traceId = status.getTraceId();
if (e == null) {
log.info("[{}] {}{} time={}ms", traceId.getId(),
addSpace(COMPLETE_PREFIX, traceId.getLevel()), status.getMessage(), resultTimeMs);
} else {
log.info("[{}] {}{} time={}ms ex={}", traceId.getId(),
addSpace(EX_PREFIX, traceId.getLevel()), status.getMessage(), resultTimeMs, e.toString());
}
releaseTraceId();
}
private void syncTraceId() {
if (traceIdHolder == null) {
traceIdHolder = new TraceId();
} else {
traceIdHolder = traceIdHolder.createNextId();
}
}
private void releaseTraceId() {
if (traceIdHolder.isFirstLevel()) {
traceIdHolder = null; //destroy
} else {
traceIdHolder = traceIdHolder.createPreviousId();
}
}
private static String addSpace(String prefix, int level) {
StringBuilder sb = new StringBuilder();
for (int i = 0; i < level; i++) {
sb.append( (i == level - 1) ? "|" + prefix : "| ");
}
return sb.toString();
}
}
TraceId
를 동기화 하는 부분만 파라미터를 사용하는 것에서 TraceId traceIdHolder
필드를 사용하도록 변경되었다.TraceId
는 파라미터로 전달되는 것이 아니라 FieldLogTrace
의 필드인 traceIdHolder
에 저장된다.syncTraceId()
TraceId
를 새로 만들거나 앞선 로그의 TraceId
를 참고해서 동기화하고, level
도 증가한다.TraceId
를 새로 만든다.TraceId
를 참고해서 동기화하고, level
도 하나 증가한다.traceIdHolder
에 보관한다.releaseTraceId()
level
이 하나 증가해야 하지만, 메서드 호출이 끝나면 level
이 하나 감소해야 한다.releaseTraceId()
는 level
을 하나 감소한다.level==0
)이면 내부에서 관리하는 traceId
를 제거한다.FieldLogTraceTest
package com.example.springadvanced.trace.logtrace;
import com.example.springadvanced.trace.TraceStatus;
import org.junit.jupiter.api.Test;
class FieldLogTraceTest {
FieldLogTrace trace = new FieldLogTrace();
@Test
void begin_end_level2() {
TraceStatus status1 = trace.begin("hello1");
TraceStatus status2 = trace.begin("hello2");
trace.end(status2);
trace.end(status1);
}
@Test
void begin_exception_level2() {
TraceStatus status1 = trace.begin("hello");
TraceStatus status2 = trace.begin("hello2");
trace.exception(status2, new IllegalStateException());
trace.exception(status1, new IllegalStateException());
}
}
level
을 통한 깊이도 잘 표현된다FieldLogTrace.traceIdHolder
필드를 사용해서 TraceId
가 잘 동기화 되는 것을 확인할 수 있다.지금까지 만든 FieldLogTrace
를 애플리케이션에 적용해보자.
스프링 빈으로 등록
LogTraceConfig
package com.example.springadvanced;
import com.example.springadvanced.trace.logtrace.FieldLogTrace;
import com.example.springadvanced.trace.logtrace.LogTrace;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
@Configuration
public class LogTraceConfig {
@Bean
public LogTrace logTrace() {
return new FieldLogTrace();
}
}
V3를 생성 했고 LogTrace
인터페이스 사용하는 걸로 코드를 변경했다.
OrderControllerV3
package com.example.springadvanced.app.v3;
import com.example.springadvanced.trace.TraceStatus;
import com.example.springadvanced.trace.logtrace.LogTrace;
import lombok.RequiredArgsConstructor;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.RestController;
@RestController
@RequiredArgsConstructor
public class OrderControllerV3 {
private final OrderServiceV3 orderService;
private final LogTrace trace;
@GetMapping("/v3/request")
public String request(String itemId) {
TraceStatus status = null;
try {
status = trace.begin("OrderController.request()");
orderService.orderItem(itemId);
trace.end(status);
return "ok";
} catch (Exception e) {
trace.exception(status, e);
throw e; //예외를 꼭 다시 던져주어야 한다.
}
}
}
애플리케이션 정상 실행
예외 실행
traceIdHolder
필드를 사용한 덕분에 파라미터 추가 없는 깔끔한 로그 추적기를 완성어떤 문제가 남아 있을까?
트랜잭션ID
가 동일하고, level
도 많이 꼬여서 나온다. 무엇이 문제일까?
FieldLogTrace
는 싱글톤으로 등록된 스프링 빈이다. 이 객체의 인스턴스가 애플리케이션에 딱 1 존재한다는 뜻이다. 이렇게 하나만 있는 인스턴스의 FieldLogTrace.traceIdHolder
필드를 여러 쓰레드가 동시에 접근하기 때문에 문제가 발생한다.
테스트 예제 코드를 통해서 동시성 문제를 확인해보자
FieldService
package com.example.springadvanced.trace.threadlocal.code;
import lombok.extern.slf4j.Slf4j;
@Slf4j
public class FieldService {
private String nameStore;
public String logic(String name) {
log.info("저장 name={} -> nameStore={}", name, nameStore);
nameStore = name;
sleep(1000);
log.info("조회 nameStore={}",nameStore);
return nameStore;
}
private void sleep(int millis) {
try {
Thread.sleep(millis);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
}
FieldServiceTest
package com.example.springadvanced.trace.threadlocal;
import com.example.springadvanced.trace.threadlocal.code.FieldService;
import lombok.extern.slf4j.Slf4j;
import org.junit.jupiter.api.Test;
@Slf4j
public class FieldServiceTest {
private FieldService fieldService = new FieldService();
@Test
void field() {
log.info("main start");
Runnable userA = () -> {
fieldService.logic("userA");
};
Runnable userB = () -> {
fieldService.logic("userB");
};
Thread threadA = new Thread(userA);
threadA.setName("thread-A");
Thread threadB = new Thread(userB);
threadB.setName("thread-B");
threadA.start(); //A실행
sleep(2000); //동시성 문제 발생X
//sleep(100); //동시성 문제 발생O
threadB.start(); //B실행
sleep(3000); //메인 쓰레드 종료 대기
log.info("main exit");
}
private void sleep(int millis) {
try {
Thread.sleep(millis);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
}
실행결과 sleep(2000)
실행 결과를 보면 문제가 없다.
Thread-A
는 userA
를 nameStore
에 저장했다.Thread-A
는 userA
를 nameStore
에서 조회했다.Thread-B
는 userB
를 nameStore
에 저장했다.Thread-B
는 userB
를 nameStore
에서 조회했다.실행결과 sleep(100)
thread-A
의 작업이 끝나기 전에 thread-B
가 실행
정리하면 다음과 같다.
1. Thread-A
는 userA
를 nameStore
에 저장했다.
2. Thread-B
는 userB
를 nameStore
에 저장했다.
3. Thread-A
는 userB
를 nameStore
에서 조회했다.
4. Thread-B
는 userB
를 nameStore
에서 조회했다.
thread-A
의 호출이 끝나면서 nameStore
의 결과를 반환받는데, 이때 nameStore
는 앞의 2번에서 userB
의 값으로 대체되었다. 따라서 기대했던 userA
의 값이 아니라 userB
의 값이 반환된다.thread-B
의 호출이 끝나면서 nameStore
의 결과인 userB
를 반환받는다.결과적으로 Thread-A
입장에서는 저장한 데이터와 조회한 데이터가 다른 문제가 발생한다. 이처럼 여러 쓰레드가 동시에 같은 인스턴스의 필드 값을 변경하면서 발생하는 문제를 동시성 문제라 한다. 이런 동시성 문제는 여러 쓰레드가 같은 인스턴스의 필드에 접근해야 하기 때문에 트래픽이 적은 상황에서는 확률상 잘 나타나지 않고, 트래픽이 점점 많아질 수 록 자주 발생한다.
- 이런 동시성 문제는 지역 변수에서는 발생하지 않는다. 지역 변수는 쓰레드마다 각각 다른 메모리 영역이 할당된다.
- 동시성 문제가 발생하는 곳은 같은 인스턴스의 필드(주로 싱글톤에서 자주 발생), 또는
static
같은 공용 필드에 접근할 때 발생한다.
그렇다면 어떻게 해결해야 될까?
이럴 때 사용하는 것이 바로 쓰레드 로컬이다.
쓰레드 로컬을 사용하면 각 쓰레드마다 별도의 내부 저장소를 제공한다. 따라서 같은 인스턴스의 쓰레드 로컬 필드에 접근해도 문제 없다.
ThreadLocalService
package com.example.springadvanced.trace.threadlocal.code;
import lombok.extern.slf4j.Slf4j;
@Slf4j
public class ThreadLocalService {
private ThreadLocal<String> nameStore = new ThreadLocal<>();
public String logic(String name) {
log.info("저장 name={} -> nameStore={}", name, nameStore.get());
nameStore.set(name);
sleep(1000);
log.info("조회 nameStore={}",nameStore.get());
return nameStore.get();
}
private void sleep(int millis) {
try {
Thread.sleep(millis);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
}
nameStore
필드가 일반 String
타입에서 ThreadLocal
을 사용하도록 변경되었다.해당 쓰레드가 쓰레드 로컬을 모두 사용하고 나면
ThreadLocal.remove()
를 호출해서 쓰레드 로컬에 저장된 값을 제거해주어야 한다.
ThreadLocalServiceTest
FieldServiceTest과 거의 비슷하다.
실행결과 sleep(100)
FieldLogTrace
에서 발생했던 동시성 문제를 ThreadLocal
로 해결해보자TraceId traceIdHolder
필드를 쓰레드 로컬을 사용하도록 ThreadLocal<TraceId> traceIdHolder
로 변경하면 된다.ThreadLocalLogTrace
package com.example.springadvanced.trace.logtrace;
import com.example.springadvanced.trace.TraceId;
import com.example.springadvanced.trace.TraceStatus;
import lombok.extern.slf4j.Slf4j;
@Slf4j
public class ThreadLocalLogTrace implements LogTrace {
private static final String START_PREFIX = "-->";
private static final String COMPLETE_PREFIX = "<--";
private static final String EX_PREFIX = "<X-";
private ThreadLocal<TraceId> traceIdHolder = new ThreadLocal<>();
@Override
public TraceStatus begin(String message) {
syncTraceId();
TraceId traceId = traceIdHolder.get();
Long startTimeMs = System.currentTimeMillis();
log.info("[{}] {}{}", traceId.getId(), addSpace(START_PREFIX, traceId.getLevel()), message);
return new TraceStatus(traceId, startTimeMs, message);
}
@Override
public void end(TraceStatus status) {
complete(status, null);
}
@Override
public void exception(TraceStatus status, Exception e) {
complete(status, e);
}
private void complete(TraceStatus status, Exception e) {
Long stopTimeMs = System.currentTimeMillis();
long resultTimeMs = stopTimeMs - status.getStartTimeMs();
TraceId traceId = status.getTraceId();
if (e == null) {
log.info("[{}] {}{} time={}ms", traceId.getId(),
addSpace(COMPLETE_PREFIX, traceId.getLevel()), status.getMessage(), resultTimeMs);
} else {
log.info("[{}] {}{} time={}ms ex={}", traceId.getId(),
addSpace(EX_PREFIX, traceId.getLevel()), status.getMessage(), resultTimeMs, e.toString());
}
releaseTraceId();
}
private void syncTraceId() {
TraceId traceId = traceIdHolder.get();
if (traceId == null) {
traceIdHolder.set(new TraceId());
} else {
traceIdHolder.set(traceId.createNextId());
}
}
private void releaseTraceId() {
TraceId traceId = traceIdHolder.get();
if (traceId.isFirstLevel()) {
traceIdHolder.remove(); //destroy
} else {
traceIdHolder.set(traceId.createPreviousId());
}
}
private static String addSpace(String prefix, int level) {
StringBuilder sb = new StringBuilder();
for (int i = 0; i < level; i++) {
sb.append( (i == level - 1) ? "|" + prefix : "| ");
}
return sb.toString();
}
}
traceIdHolder
가 필드에서ThreadLocal
로 변경되었다. 따라서 값을 저장할 때는 set(..)
을 사용하고, 값을 조회할 때는 get()
을 사용한다.ThreadLocal.remove()
를 호출해서 쓰레드 로컬에 저장된 값을 제거해주어야 한다. ThreadLocalLogTraceTest
package com.example.springadvanced.trace.logtrace;
import com.example.springadvanced.trace.TraceStatus;
import lombok.extern.slf4j.Slf4j;
import org.junit.jupiter.api.Test;
@Slf4j
class ThreadLocalLogTraceTest {
ThreadLocalLogTrace trace = new ThreadLocalLogTrace();
@Test
void begin_end_level2() {
TraceStatus status1 = trace.begin("hello1");
TraceStatus status2 = trace.begin("hello2");
trace.end(status2);
trace.end(status1);
}
@Test
void begin_exception_level2() {
TraceStatus status1 = trace.begin("hello");
TraceStatus status2 = trace.begin("hello2");
trace.exception(status2, new IllegalStateException());
trace.exception(status1, new IllegalStateException());
}
}
애플리케이션에 적용해보자.
LogTraceConfig - 수정
package com.example.springadvanced;
import com.example.springadvanced.trace.logtrace.LogTrace;
import com.example.springadvanced.trace.logtrace.ThreadLocalLogTrace;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
@Configuration
public class LogTraceConfig {
@Bean
public LogTrace logTrace() {
//return new FieldLogTrace();
return new ThreadLocalLogTrace();
}
}
동시성 문제 확인을 위해(localhost:8080/v3/request?itemId=hello)을 연속으로 해보자.
트랜잭션ID
와 level
이 정상적으로 나오는 것을 확인 할 수 있다.쓰레드 로컬의 값을 사용 후 제거하지 않고 그냥 두면 WAS(톰캣)처럼 쓰레드 풀을 사용하는 경우에 심각한 문제가 발생할 수 있다.
사용자A 저장 요청
1. 사용자A가 저장 HTTP를 요청했다.
2. WAS는 쓰레드 풀에서 쓰레드를 하나 조회한다.
3. 쓰레드 thread-A
가 할당되었다.
4. thread-A
는 사용자A 의 데이터를 쓰레드 로컬에 저장한다.
5. 쓰레드 로컬의 thread-A
전용 보관소에 사용자A 데이터를 보관한다.
사용자A 저장 요청 종료
1. 사용자A의 HTTP 응답이 끝난다.
2. WAS는 사용이 끝난 thread-A
를 쓰레드 풀에 반환한다. 쓰레드를 생성하는 비용은 비싸기 때문에 쓰레드를 제거하지 않고, 보통 쓰레드 풀을 통해서 쓰레드를 재사용한다.
3. thread-A
는 쓰레드풀에 아직 살아있다. 따라서 쓰레드 로컬의 thread-A
전용 보관소에 사용자A 데이터도 함께 살아있게 된다.
사용자B 조희 요청
1. 사용자B가 조회를 위한 새로운 HTTP 요청을 한다.
2. WAS는 쓰레드 풀에서 쓰레드를 하나 조회한다.
3. 쓰레드 thread-A
가 할당되었다. (물론 다른 쓰레드가 할당될 수 도 있다.)
4. 이번에는 조회하는 요청이다. thread-A
는 쓰레드 로컬에서 데이터를 조회한다.
5. 쓰레드 로컬은 thread-A
전용 보관소에 있는 사용자A 값을 반환한다.
6. 결과적으로 사용자A 값이 반환된다.
7. 사용자B는 사용자A의 정보를 조회하게 된다.
이런 문제를 예방하려면 사용자A의 요청이 끝날 때 쓰레드 로컬의 값을
ThreadLocal.remove()
를 통해서 꼭 제거해야 한다.
참고
김영한: 스프링 핵심 원리 - 고급편(인프런)
Github - https://github.com/b2b2004/Spring_ex