You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by "balloon72 (via GitHub)" <gi...@apache.org> on 2024/03/29 02:33:11 UTC

[PR] [INLONG-9884][Manager] Optimized code AuditServiceImpl [inlong]

balloon72 opened a new pull request, #9896:
URL: https://github.com/apache/inlong/pull/9896

   Title Name: [INLONG-9884][Manager] Optimized code AuditServiceImpl
   
   https://github.com/apache/inlong/issues/9884
   
   Motivation
   Explain here the context, and why you're making that change. What is the problem you're trying to solve.
   
   Modifications
   Describe the modifications you've done.


-- 
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: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [INLONG-9884][Manager] Optimized code AuditServiceImpl [inlong]

Posted by "balloon72 (via GitHub)" <gi...@apache.org>.
balloon72 commented on PR #9896:
URL: https://github.com/apache/inlong/pull/9896#issuecomment-2026573909

   Great idea, I completely agree with this approach. Next time, I will create an umbrella issue to track code optimization problems.


-- 
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: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [INLONG-9884][Manager] Optimized code AuditServiceImpl [inlong]

Posted by "dockerzhang (via GitHub)" <gi...@apache.org>.
dockerzhang merged PR #9896:
URL: https://github.com/apache/inlong/pull/9896


-- 
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: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [INLONG-9884][Manager] Optimized code AuditServiceImpl [inlong]

Posted by "aloyszhang (via GitHub)" <gi...@apache.org>.
aloyszhang commented on PR #9896:
URL: https://github.com/apache/inlong/pull/9896#issuecomment-2026529790

   @balloon72 Thanks for your contribution!
   This is a code-optimization type pull request same as #9888. Other places may also have the same problem. There will be too many issues and pull requests if we resolve them individually, also, it's hard to track all the issues.
   
   So, I recommend bringing an umbrella issue to track all the code optimization issues at the module or sub-module level, each module or sub-module has only one issue to deal with the code optimization.
   WDYT? Please feel free to share your opinions.


-- 
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: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org