ruanwenjun commented on PR #15730:
URL: 
https://github.com/apache/dolphinscheduler/pull/15730#issuecomment-2004207592

   > I have a question not directly related to this PR. Since currently we 
encapsulate the whole query stuff in Mappers, do we still need DAOs? It seems 
our DAOs have already lost the function of decoupling.
   
   it's better to return DO in the Mapper, and return DTO in DAO. So the DAO 
might contain some merge or transform function.


-- 
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