Spring Code Refactoring - 구조 설계 방법

Question

안녕하세요. 선배님들지금 현재 회사에서 프로젝트를 진행중인데, 코드를 리팩토링하는 과정중에 아무리 머리를 굴려도 답이 안보여서 이렇게 글을 남깁니다. 현재 A서비스 인터페이스와 A서비스를 상속받은 Aimpl이 있습니다.Aservice.java// Aservice.java public interface Aservice { public void updateInfo(Object info) }AserviceImple.java // AserviceImpl.java &#64;Service(Aservice) public class AserviceImpl implements Aservice { &#64;Autowired public BusinessDao BusinessDao; &#64;Override public void updateInfo(Object info) { // ...별도의 로직 return BusinessDao.updateQuery(info); } } 다른 BserviceImpl에서 A라는 서비스의 updateInfo를 사용하고 싶어서BserviceImple내에 A서비스 의존성 주입(?)을 하고 사용하려고 했습니다. // BserviceImple.java public class BserviceImpl implements Bservice { &#64;Autowired private Aservice Aservice; &#64;Override pubic void updateBinfo(Object info) { // ....별도의 로직 Aservice.updateInfo(info); } } 선임자분이 그렇게할 경우Aservice의 updateinfo와 Bservcie의 updateBinfo를 다른 사용자들이 동시에 호출할 경우Aservice의 트랜잭션과 Bservice트랜잭션이 중첩되어서,DB Lock이 걸릴 수도 있다고 지양하라고 하더라구요.. 제가 트렌잭션 전파에 대해 설명을 해도 이론은 그렇지 실무에서는 예외사항이 발생하고그런 경우를 경험했다는 말에 더이상 할말은 없고, 일단 수정을 해보겠다고 했습니다. 그래서 이 코드를 리팩토링을 하고싶은데 어떻게 구조를 가져가야할지 막막합니다.선임자분은 Aservice의 updateInfo 메소드를 B에도 만들어라는 식으로 말씀을 하시는데있는 코드를 재사용을 하면 되는 걸 굳이 불필요하게 또 해야할지.. 어떤식으로 리팩토링을 해야할지 조언을 주시면 감사하겠습니다 선배님들!

LichKing · Answer

A 객체를 주입받아호출하면 문제가 되고 동일한 내용을 B에도 만들면 문제가 안된다고요??이해하기가 힘드네요

피맥 · Answer

LichKing저가 일단 빼먹은 부분이 있는데 수정하겠습니다.동시에 A서비스의 updateinfo를 호출하고, B서비스의 updateBinfo를 호출할 때문제가 발생한다는 것이였습니다.그런 문제는 이해를 하지만 왜 Service에서 Service를 호출하면 안되는지 이해가 안되어서, 결국 그 문제는 Service호출 문제가 아니라같은 DAO를 쓰면 문제가 되는거 아니냐 이런식으로 다시 여쭤보았습니다. 그랬더니 트렌젝션은 서비스단에 걸리는 것이기 때문에 이게동시에 A서비스의 updateinfo를 호출하고, B서비스의 updateBinfo를 호출하면DB Lock이 걸린다고 하지말라고 하네요..계속 힌트라도 주면 고치겠는데, 힌트가 아니라 다른 이야기를 하니깐 답답합니다..

더미 · Answer

트랜잭션을 dao에 걸면 되죠.

LichKing · Answer

트랜잭션걸려있는 메서드가 다른 트랜잭션메서드를 호출하면 스프링 트랜잭션 기본설정이 상위 트랜잭션에 포함되게뜸 되어있습니다. 이는 아마 트랜잭션 고립성 설정같은걸 통해 바꿀수있을텐데요. &#64;Transactional 애노테이션에 인자를 넘겨서 활용가능하구요. 그런데 디폴트가 이미 시작된 트랜잭션이 있을경우 그 트랜잭션에 포함되는거라 따로 설정 주셔야할게 없습니다. 빼먹은부분을 추가하셔도 이해안되는건 마찬가지네요.

피맥 · Answer

LichKing답변감사합니다..! 저도 고립성을 이용할까 했는데 무조건 DB Lock걸린다는 말만 하시니깐저도 답답합니다ㅠㅠ 선임자분이 경험을 했다고하니깐 반박은 못하겠고선임자분은 동시에 실행시킬때 같은 key값을 동시에 수정하려고 할때 어떻게 할꺼냐이런말 같은데...혹시 다른 말이지만 service에서 다른 service를 호출하는 건 크게 문제가 없는건가요? 지양하는 방법인지도 알고싶습니다..!아니면 저런 구조일때 보편적으로 선배님은 어떤식으로 리팩토링을 하는지도 여쭤봐도 될지요.

피맥 · Answer

더미기본설정은 일단 impl이간 한데,한번 여쭤봐야겠네요..

LichKing · Answer

서비스에서 서비스 호출하는건 매우 일상적이죠 ㅎㅎ

B급 개발자 · Answer

선임분이 살짝 오해가 있으신거 같네요 (물론 제가 오해하는 것일 수도...)1. DB Lock 은 DB에서 걸리는 것입니다. (어플리케이션에서) 트랜잭션 설정을 어떻게 하였든 간에 (DB에서) 특정 세션이 사용하고 있는 리소스에 다른 세션이 접근하려 하면 막힙니다. 이것이 write 또는 read 작업에 걸릴지 table 또는 특정 row에만 걸릴지는 DBMS 제품에 따라 다릅니다. 2. 트랜잭션은 동일 Thread에서만 유효하며 각각의 Thread에서는 서로 Safe 합니다. 이 얘기는 쉬운 말로 같은 사용자의 같은 호출일 경우에는 트랜젝션의 처리가 유효하지만 순차적으로 실행되는게 맞고 동시적으로 발생된 다른 호출의 경우에는 트랜잭션이 유효하지 않다는 것입니다. 다만 고려해야 하는 것은 해당 트랜잭션이 장시간 실행되는 것이라면 위 1의 경우에 따라 Lock 이 걸릴 것입니다. 하지만 이것은 로직을 개선해야 할 문제이지 트랜잭션 설정과는 좀 무관해 보입니다.

대빵 · Answer

선배한테 이렇게 얘기하세여....트랜잭션은 자바 어플리케이션에서 제어하는게 아니고 DB 에서 하는 거라고...자바는 그냥 정해진데로 코딩만 하는거지 실제로 제어는 DB 가 하는 거라고...A update 진행도중 B update 가 호출되면 DB 는 B를 locking 합니다....그런데 이게 다가 아니져...A 가 끝나면 B 의 lock 은 어떻게 될까여? isolation 정책에 따라 다르지만 lock 해제되면서 B도 수행되는 경우가 대부분 입니다.(특별한 where절에 변화가 없다면....)즉, row level 에서 locking을 지원하는 경우 (오라클) 동일 row 만 동시에 update 하지 않으면 문제가 없는 거고 만일 동일한 row 를 서로 다른 프로그램에서 동시에 update 하게 구현되었다면 설계문제가 거의 확실합니다(데이터에 대한 CUD 권한문제)결론적으로, 트랜잭션에는 문제가 없습니다......dead lock 이 문제인거고 dead lock은 설계변경으로 해결해야 합니다.

스텁 · Answer

디비 관점에서 보면 한 프로세스에서 처리할것을 트랜잭션 쪼개버리면 오히려 락 더 걸리고요 반대로 한개의 업무단위도 아닌데 불필요하게 같은 트랜잭션으로 묶어버리면 처리 시간이 그만큼 길어지기 때문에 또다른 요청이 들어올때 락이 걸릴수 있게 되서 묶으면 안되는건데요 LichKing 님 말씀처럼 상위 메소드로 트랜잭션이 합쳐져야 하는게 디폴트여야 합니다. 그래야 한개의 요청안에서는 서로 락이 안걸리게 되니까요. 선임분이 이야기 하신거보면 작성자님네 서비스가 이미 트랜잭션 분리하도록 설정되어 있을수도 있지 않나요? 단순히 서비스 메소드 단위로 트랜잭션이 걸리게 되어있다고 해서 합치지 말라고 한것보다는 업무적으로 분리해야 하는 트랜잭션이라서 그렇게 이야기 한거여야 할거 같네요.

피맥 · Answer

램그로프음 현제 오라클을 쓰고 있고,저의 생각도 램그로프님이랑 비슷합니다.위에도 쓰여있지만 결국 같은 key값을 수정할 때 문제기 때문에 이건 같은 DAO쿼리를 쓰면문제이지 않냐고 했지만, 무조건 service 트렌젝션 탓을 하네요..A트렌젝션이 있지만 B트렌젝션에서 중첩될 경우 순차적으로 풀려야하는데 안풀릴 수가 있다고..답답합니다 정말..

피맥 · Answer

대빵오라클에 대해 설명하면서 오라클은 row로 lock이 걸린다고 해도,버전에 따라 다르고 그 해당 row뿐 아니라 다른 row도 걸릴수 있다고 애매모호하게 말하는데..그럼 혹시 동일한 row를 막기 위한 설계의 간단한 예시나 어떤식으로 하는지 해주실 수 있으실까요 선배님..아니면 간단한 검색 키워드라도 알려주시면 감사하겠습니다..

피맥 · Answer

스텁음 현재 서비스 메소드 특정 접두어 별로 트랜잭션이 자동으로 설정되어있습니다. 위에 예시로 든 A서비스가 데이터를 단순 정리하여 업데이트를 하는 메소드라면B서비스에서는 데이터 한번도 조합&#43;정리를 하는 메소드에 A서비스와 같은 기능인 데이터 정리하여 업데이트하는 로직입니다. 정리하면 B서비스에서 A서비스를 부르기전에 조합 정리하는 중에도 한번 DB수정이 일어나서 같은 트렌젝션단위로 묶기위해 써놓은 상황입니다. 그래서 다른 트렌젝션단위로 분리한다는 것도 힘든 상황인데..혹시 이렇게 될 경우보편적인 설계방법도 궁금하네요..

피맥 · Answer

LichKing감사합니다! 요즘 다시한번 공부하는 중인데, 이론과 실제가 달라서 많이 힘드네요..ㅠㅠ아 그리고 죄송하지만 마지막 질문 한가지만 드려도 될까요? 저가 A서비스 메소드 중 처리할게 많아서A서비스 내에서 자체적으로 메소드를 뽑아서 만든것이 있습니다예를 들면public class AserviceImpl implements Aservice { &#64;Override public void UpdateMethod(Object info) { // ...로직 copyA(info) //..로직 } public void copyA(Object info) { bussinessDAO.update(info) } } 이런식으로 짜놨는데.. 이런 경우도 트렌젝션 문제가 생길 수 있다고 메소드로 빼지말고 그냥 쓰라고 하는데이럴 경우도 문제가 생기나요? 도무지 이해가 안됩니다..저가 잘못알고 있는건지책에 있는 내용을 그대로 설명을 해도 그건 가이드일뿐이지 실전에서는 그렇게 되지 않는다고 하니답답합니다..

대빵 · Answer

아직 정확하게 이해를 못하신것 같은데, lock 이 걸리는게 문제가 아닙니다. lock은 필요할때 당연히 걸리는게 정상이구여..lock 이 정확하게 걸려주기 때문에 비싼 돈 주고 db를 사서 쓰는 거져....문제는 dead lock 즉, 양쪽에서 lock을 잡고 동시에 교착상태로 빠져서 lock 이 풀리지 않는게 문제인 거구여.이 dead lock은 기능상의 문제가 아니라는 겁니다. dead lock은 설계상의 문제 입니다.데이터에 대한 제어 권한이 두개의 모듈에 나눠서 있는 상태에서 각각 서로 다르게 작동하게 설계하고서 dead lock 이 걸릴 확률이 있다고 얘기하면 그건 설계가 잘못된 거져...만일 설계를 정확하게 해서 dead lock을 방지하고 싶다면 BserviceImpl 에서 A를 주입할 것이 아니고 BserviceImple 자체를 직접 호출하고 해당 method를 synchronized 걸어서 동시에 두개의 thread 가 접근하지 못하게 막는 설계를 하고 구현을 해야 되는 거지 dead lock 방지 및 회피(데이터 수정에 대한 owner ship 이 Bserviceimple 에도 존재하는 것이 맞는 설계인가에 대한 고민 및 의사결정)에 대한 아무런 설계 없이 그냥 머리속으로 구현하다가 갑자기 옆에서 보고 그렇게 하면 dead lock 이 생길수도 있다니.......그건 그냥 건물 건축할때 설계에 대한 고민 없이 그냥 대충 건물 짓다가 이렇게 지으면 건물 무너질수도 있다고 말하는 것과 같은 겁니다......선배가 설계담당자라면 명확한 설계스펙을 먼저 고민해야 되는 거고 , 본인이 설계담당자라면 data ownership 에 대한 정확한 R&R 과 class 및 entity 에 대한 ORM 을 명확히 정의한 상태에서 각각의 service 들의 역할을 어디까지 줄것 인가를 명확히 하고 구현 진행해야 합니다.쓰고 보니 장황하지만 결국 해당 서비스가 데이터를 수정하는 것이 맞는 건가? 의 의사결정 문제 입니다.

피맥 · Answer

대빵좋은 조언 감사드립니다!저가 알고싶어하는 부분을 정확히 알려주신것 같습니다..!결국 A서비스도 따로 데이터를 수정해야하고, B서비스에서도 똑같이 수정해야하는데A와 B에게 어느쪽이 우선순위이고 어디까지 할 것인가를 정확히 명시해달라고 해야겠습니다.선배님 답글을 쓰다가 궁금한게 생겼는데,현재 synchronized에 대해 검색을 하고, 자바 기본서적에서 보다가 드는 생각인데혹시 A서비스 메소드에 비동기를 걸고, B서비스에서 A서비스를 호출을 했습니다.그럼 1번사람은 A서비스 메소드를 실행하고, 2번 사람은 B서비스를 동시에 실행을 할 때1번사람의 A서비스가 먼저 처리중 B서비스가 처리하다가 A서비스 호출을 할 때, 이때는 1번 사람의 A서비스를 처리하고, B서비스에서 호출한 A서비스가 작동하는 지가 궁금합니다.실제로 해보지 않고, 이렇게 먼저 질문글을 남긴 점 죄송합니다..

대빵 · Answer

실제 코드를 보지 않고 명확히 할수 없는 사항이지만만일 synchronized 걸려있는 함수에 1번사람이 요청한 사항이 먼저 진입한다면 2번사람은 기다리게 됩니다.단지 1번사람이 비동기로 호출했다면 1번사람이 먼저 synchronized 함수에 진입할수 없을 수도 있습니다.synchronized 는 멀티쓰레드에서 동시에 한개의 쓰레드만 허용하는 구문입니다. 이걸 의심하면 사실 개발하기 힘들어져염.....ㅎㅎ

Spring Code Refactoring - 구조 설계 방법

17개의 답변

Q&A 목록