[GitHub] [incubator-celeborn] nafiyAix opened a new pull request, #1005: [CELEBORN-58][REFACTOR] Aggregate reserve failed logs together

[GitBox <gi...@apache.org>]

nafiyAix opened a new pull request, #1005:
URL: https://github.com/apache/incubator-celeborn/pull/1005

   ### What changes were proposed in this pull request?
   Aggregate reserve failed logs together
   
   ### Why are the changes needed?
   Avoid to print too many log when fail to reserve slots.
   
   ### What are the items that need reviewer attention?
   The log will looks like , there will be dot at the end of each error statement in real situation.
   <img width="399" alt="image" src="https://user-images.githubusercontent.com/30563796/203972214-886ba53a-0909-4696-9c85-549fe6d76313.png">
   
   
   ### Related issues.
   [CELEBORN-58](https://issues.apache.org/jira/browse/CELEBORN-58)
   
   ### Related pull requests.
   
   
   ### How was this patch tested?
   
   
   /cc @AngersZhuuuu 
   
   /assign @main-reviewer
   


-- 
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: dev-unsubscribe@celeborn.apache.org

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

[GitHub] [incubator-celeborn] waitinfuture merged pull request #1005: [CELEBORN-58][REFACTOR] Aggregate reserve failed logs together

[GitBox <gi...@apache.org>]

waitinfuture merged PR #1005:
URL: https://github.com/apache/incubator-celeborn/pull/1005


-- 
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: dev-unsubscribe@celeborn.apache.org

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