Re: [PR] [Bug][Master]serial_wait strategy workflow unable to wake up [dolphinscheduler]

Fri, 12 Jan 2024 00:41:34 -0800 


fuchanghai commented on PR #15270:
URL: 
https://github.com/apache/dolphinscheduler/pull/15270#issuecomment-1888663688

   > > > hi @Gallardot I remember that for nested things, you need to use a 
proxy object to make it effective. and need add ` EnableAspectJAutoProxy` on 
Application cc @ruanwenjun
   > > 
   > > 
   > > Which part does it mean?
   > 
   > <img alt="image" width="1005" 
src="https://private-user-images.githubusercontent.com/33984497/296174772-17ed73ef-279e-47a3-b1d4-449fa45fe6cb.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDUwNDg5OTAsIm5iZiI6MTcwNTA0ODY5MCwicGF0aCI6Ii8zMzk4NDQ5Ny8yOTYxNzQ3NzItMTdlZDczZWYtMjc5ZS00N2EzLWIxZDQtNDQ5ZmE0NWZlNmNiLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAxMTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMTEyVDA4MzgxMFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWYyMTJhZmNhMGViOTAxNWZjNjQ5ODI1MzdjOGQ5MzQ2OTgzN2UzODVkMjFhODI5YzE5ODI1ZDBkOTU3MDRiMjMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.yxaTGHjwgaE93baspRcHRkWYA80pkrOSP_yRgI8U1W8";>
 The transaction has been opened on the handleCommand method, and I am not sure 
whether the new transaction in this method will take effec
 t. AFAIK, this is a nested transaction and needs to be enabled for annotations 
to take effect. ```@ EnableAspectJAutoProxy(exposeProxy = true) ```,Maybe I'm 
wrong, can you test whether the database data is updated after running the 
upsert method? cc @EricGao888 @ruanwenjun
   
   @Gallardot Sorry I didn't notice EnableTransactionManagement, and LGTM


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to