[JDBC 라이브러리 구현하기 - 3, 4단계] 코코닥(이승용) 미션 제출합니다. #541

kokodak · 2023-10-08T20:01:59Z

안녕하세요 파워! 길었던 연휴도 이제 슬슬 끝나가네요. 파워는 컨디션 회복 많이 하셨나요?

이번 미션 구현이 제 우선순위에서 많이 밀려서 시간을 많이 못썼습니다 🥲
그래서 3, 4단계를 한번에 구현해서 제출하게 되었습니다 ㅠㅠ..

우선 3단계 요구사항이었던 로직의 원자성을 보장하기 위한 트랜잭션 적용 과, 4단계 요구사항이었던 트랜잭션 동기화, 서비스 추상화 모두 만족시킨 것 같습니다.

코드의 대략적인 흐름은 아래 그래프와 같습니다.

graph LR
TransactionTemplate --> TransactionManager
TransactionManager --> TransactionSynchronizationManager

graph LR
Dao --> JdbcTemplate
JdbcTemplate --> DataSourceUtils
DataSourceUtils --> TransactionManager
DataSourceUtils --> TransactionSynchronizationManager

TransactionTemplate 은 트랜잭션을 적용하면서 발생하는 중복 로직을 제거하기 위해 만들어진 템플릿 콜백 클래스입니다.

TransactionManager 에서는 트랜잭션 시작, 커밋, 롤백 등을 담당하고 있습니다. 각 스레드 및 데이터소스에서 진행중인 트랜잭션이 존재하는지 확인해주기 위해 ThreadLocal 로 해당 상태를 관리해주기도 합니다.

DataSourceUtils 에서는 Connection 생성/파괴를 관리하는데, 진행중인 트랜잭션이 있다면 Connection 을 재활용할 수 있도록 도와줍니다.

추가로, 분산 트랜잭션이나 트랜잭션 전파에 대해서는 고려하지 않았습니다!

리뷰 잘 부탁드립니다!!

- TransactionManager 는 트랜잭션의 시작, 커밋, 롤백 등의 로직을 담당한다.

- DataSourceUtils 는 트랜잭션 동기화 기능을 사용하여, 같은 Connection 객체를 재사용할 수 있도록 한다.

…ion 을 다루도록 변경

thdwoqor

안녕하세요 코코닥
3, 4단계 구현해주신다고 고생많으셨습니다
코멘트 조금 남겼으니 확인부탁드려요 🙂

thdwoqor · 2023-10-09T12:57:31Z

jdbc/src/main/java/org/springframework/transaction/support/TransactionTemplate.java

+
+    private final TransactionManager transactionManager;
+
+    public TransactionTemplate(final TransactionManager transactionManager) {


사용하지않는 생성자인거 같아요

삭제하겠습니다!

thdwoqor · 2023-10-09T13:25:37Z

jdbc/src/main/java/org/springframework/jdbc/datasource/DataSourceUtils.java

-            TransactionSynchronizationManager.bindResource(dataSource, connection);
+            final Connection connection = dataSource.getConnection();
+            if (TransactionManager.isTxBegin(dataSource)) {
+                connection.setAutoCommit(false);


DataSourceUtils에서
connection.setAutoCommit(false); 를 하는 이유가 무엇인가요?
트랜잭션을 사용하지 않는 부분도 전부 setAutoCommit(false)가 되는거같은데
transactionManager.begin(); 시에 false 해야하지않을까요?

DM 나눴던 내용으로 리팩터링 수행해보겠습니다!

thdwoqor · 2023-10-09T13:27:39Z

jdbc/src/main/java/org/springframework/transaction/support/TransactionManager.java

+    }
+
+    public static boolean isTxBegin(final DataSource dataSource) {
+        final Boolean flag = isTxBegin.get().get(dataSource);


connection.getAutoCommit(); 메서드를 활용할수있을것같아요

이런 메서드가 있었군요...! 알려주셔서 감사합니다.

다만 문제가 됐던 지점이 있었는데, 해당 메서드를 사용하려면 파라미터로 넘어오는 DataSource 을 기반으로 Connection 을 찾아와야 합니다.

해당 부분은 DataSourceUtils.getConnection() 메서드를 사용해서 Connection 을 찾아올 순 있으나, 문제는 isTxBegin() 메서드가 public static 으로 열려 있기 때문에, 자칫 잘못 사용하면 새로운 Connection 객체를 새로 만들어서 가져오고, 그것에 대한 getAutoCommit() 을 사용하기 때문에.. 필요없는 성능 비용 지출이 생길 위험이 있었습니다.

때문에 현재 열려있는 트랜잭션이 존재하는가? 에 대한 판단을 효율적으로 하기 위해선, 트랜잭션이 시작됐을 때 그에 대한 flag 값을 어디엔가 캐싱해두는 작업이 필요합니다. 이를 위해서 threadLocal 변수로 따로 빼두었던 것인데, 생각해보니 트랜잭션이 시작됐을 때 그에 대한 flag 값 은 TransactionSynchronizationManager 에 있는 resources 변수를 통해 이미 관리가 되고 있었더라구요. (TransactionSynchronizationManager 내부 resources 맵 안에는 트랜잭션이 시작된 Connection 에 대해서만 저장하고 있기 때문에)

따라서 TransactionManager 내부의 ThreadLocal 변수를 삭제하고, 열려있는 트랜잭션이 존재하는가? 에 대한 판단 책임을 TransactionManager 에서 TransactionSynchronizationManager 로 옮겼습니다!

thdwoqor · 2023-10-09T13:35:42Z

...src/main/java/org/springframework/transaction/support/TransactionSynchronizationManager.java


 public abstract class TransactionSynchronizationManager {

-    private static final ThreadLocal<Map<DataSource, Connection>> resources = new ThreadLocal<>();
+    private static final ThreadLocal<Map<DataSource, Connection>> resources = ThreadLocal.withInitial(HashMap::new);


withInitial 라는 메서드가 있었군요 👍

thdwoqor

변경사항 확인했습니다~!
미션 하느라 고생하셨어요!
다음 미션도 화이팅 👍

kokodak added 10 commits October 5, 2023 20:31

refactor: UserHistoryDao 가 JdbcTemplate 을 사용하도록 리팩터링

7b27d07

feat: TransactionSynchronizationManager 내부 로직 구현

0890fba

feat: TransactionManager 클래스 구현

ac8250f

- TransactionManager 는 트랜잭션의 시작, 커밋, 롤백 등의 로직을 담당한다.

feat: DataSourceUtils 내부 로직 구현

953d7d8

- DataSourceUtils 는 트랜잭션 동기화 기능을 사용하여, 같은 Connection 객체를 재사용할 수 있도록 한다.

refactor: PreparedStatementExecutor 에서 DataSourceUtils 를 사용하여 Connect…

bff3241

…ion 을 다루도록 변경

refactor: UserService 추상화 및 비즈니스 로직과 트랜잭션 로직 구현체 분리

d96e3f0

chore: 네이밍 변경

2f5aaa9

feat: 중복 코드 제거를 위한 트랜잭션 템플릿 콜백 클래스 생성

3e5b598

refactor: 템플릿 콜백 패턴 적용

6b3aaf6

refactor: PreparedStatementExecutor.execute 메서드에 가변인자 파라미터를 넘기지 않도록 수정

162465b

kokodak self-assigned this Oct 8, 2023

kokodak requested a review from thdwoqor October 8, 2023 20:02

thdwoqor requested changes Oct 9, 2023

View reviewed changes

kokodak added 2 commits October 10, 2023 14:45

refactor: Connection.setAutoCommit 옵션 설정의 책임을 TransactionManager 로 이동

16398f8

refactor: 사용하지 않는 생성자 삭제

bdd06c8

thdwoqor approved these changes Oct 10, 2023

View reviewed changes

thdwoqor merged commit bc3b5bb into woowacourse:kokodak Oct 10, 2023
1 check failed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[JDBC 라이브러리 구현하기 - 3, 4단계] 코코닥(이승용) 미션 제출합니다. #541

[JDBC 라이브러리 구현하기 - 3, 4단계] 코코닥(이승용) 미션 제출합니다. #541

kokodak commented Oct 8, 2023

thdwoqor left a comment

thdwoqor Oct 9, 2023

kokodak Oct 10, 2023

thdwoqor Oct 9, 2023

kokodak Oct 10, 2023

thdwoqor Oct 9, 2023

kokodak Oct 10, 2023

thdwoqor Oct 9, 2023

thdwoqor left a comment


		private final TransactionManager transactionManager;

		public TransactionTemplate(final TransactionManager transactionManager) {

[JDBC 라이브러리 구현하기 - 3, 4단계] 코코닥(이승용) 미션 제출합니다. #541

[JDBC 라이브러리 구현하기 - 3, 4단계] 코코닥(이승용) 미션 제출합니다. #541

Conversation

kokodak commented Oct 8, 2023

thdwoqor left a comment

Choose a reason for hiding this comment

thdwoqor Oct 9, 2023

Choose a reason for hiding this comment

kokodak Oct 10, 2023

Choose a reason for hiding this comment

thdwoqor Oct 9, 2023

Choose a reason for hiding this comment

kokodak Oct 10, 2023

Choose a reason for hiding this comment

thdwoqor Oct 9, 2023

Choose a reason for hiding this comment

kokodak Oct 10, 2023

Choose a reason for hiding this comment

thdwoqor Oct 9, 2023

Choose a reason for hiding this comment

thdwoqor left a comment

Choose a reason for hiding this comment