In a Spring Boot service, a fresher writes the following code: @Service public class OrderService { public void processAll(List orders) { for (Order o : orders) { saveOne(o); // direct call to method in the same class } } @Transactional public void saveOne(Order o) { orderRepository.save(o); } } When processAll() is called from a controller and saveOne() throws a RuntimeException midway, the already-saved rows are NOT rolled back. What is the main reason?

Question

In a Spring Boot service, a fresher writes the following code:

@Service
public class OrderService {
    public void processAll(List<Order> orders) {
        for (Order o : orders) {
            saveOne(o); // direct call to method in the same class
        }
    }

@Transactional
    public void saveOne(Order o) {
        orderRepository.save(o);
    }
}

When processAll() is called from a controller and saveOne() throws a RuntimeException midway, the already-saved rows are NOT rolled back. What is the main reason?

Accepted Answer

Spring triển khai @Transactional bằng proxy (JDK dynamic proxy hoặc CGLIB): chỉ khi lời gọi đi *qua* proxy từ bên ngoài bean thì advice mở/commit/rollback transaction mới chạy. Gọi saveOne() trực tiếp trong cùng class dùng tham chiếu `this`, nên hoàn toàn bỏ qua proxy. Đáp án (a) sai ngược: mặc định Spring rollback với unchecked exception (RuntimeException/Error), còn checked exception mới không rollback; (c) là bịa đặt, vòng lặp không ảnh hưởng gì; (d) sai vì propagation mặc định đã là REQUIRED, không phải NOT_SUPPORTED. Cách sửa: tách saveOne sang bean khác, hoặc tự inject proxy / dùng TransactionTemplate.

Câu hỏi hôm nay