Hi Cherry， Thanks for sharing this information with us. Normally we just need make sure @Transactional Aspect is last one to the wrapper. Can you send a PR by clean up the order thing?
Willem Jiang Twitter: willemjiang Weibo: 姜宁willem On Mon, Sep 17, 2018 at 12:21 PM cherrylzhao <[email protected]> wrote: > > Hi > > I found that @order annotation can’t work in TransactionAspectConfig.java. > Now, we define the @order(0), @order(1) in Aspect beans, but when I adjust > the value of order, > The aspect chains of dynamic proxy bean can’t change. > > It seems that @order annotation can work with @Aspect annotation, so we need > move the @order to the head of aspect bean like this: > ----- > @Aspect > @Order(0) > public class SagaStartAspect {} > > But this need omega-transaction module add spring dependency，it will increase > the coupling of omega module. > Also we can remove the @order annotation from TransactionAspectConfig.java, > so the default order is lowest and SagaStartAspect will be inside the > @Transactional. > > > Best Wishes & Regards > >
