You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@uniffle.apache.org by GitBox <gi...@apache.org> on 2022/08/04 14:25:58 UTC

[GitHub] [incubator-uniffle] smallzhongfeng opened a new issue, #127: [Improvement] Add timeout reconnection in delegationrssshufflemanager

smallzhongfeng opened a new issue, #127:
URL: https://github.com/apache/incubator-uniffle/issues/127

   When `DelegationRssShuffleManager` send an `AccessClusterRequest` to the Coordinator, should we increase the logic of overtime resending?


-- 
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@uniffle.apache.org.apache.org

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


[GitHub] [incubator-uniffle] smallzhongfeng commented on issue #127: [Improvement] Add timeout reconnection in DelegationRssShuffleManager

Posted by GitBox <gi...@apache.org>.
smallzhongfeng commented on issue #127:
URL: https://github.com/apache/incubator-uniffle/issues/127#issuecomment-1206024409

   Sure, I am trying to do this.


-- 
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@uniffle.apache.org

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


[GitHub] [incubator-uniffle] smallzhongfeng commented on issue #127: [Improvement] Add timeout reconnection in DelegationRssShuffleManager

Posted by GitBox <gi...@apache.org>.
smallzhongfeng commented on issue #127:
URL: https://github.com/apache/incubator-uniffle/issues/127#issuecomment-1205999294

   The reason is that the `ShuffleServer` returns the heartbeat at a frequency of 10s each time. The `Coordinator` filters the `ShuffleServer` whose available memory is lower than the threshold value, which may cause the `ShuffleServer` to be lower than the allocable quantity. In this way, it is pushed down to SSM, add a retry logic and two parameters, retry times and time. Within the user's expected retry time, run on RSS as much as possible.WDYT?


-- 
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@uniffle.apache.org

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


[GitHub] [incubator-uniffle] jerqi commented on issue #127: [Improvement] Add timeout reconnection in delegationrssshufflemanager

Posted by GitBox <gi...@apache.org>.
jerqi commented on issue #127:
URL: https://github.com/apache/incubator-uniffle/issues/127#issuecomment-1205381611

   > 
   
   Why do we need this? What's your situation? 


-- 
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@uniffle.apache.org

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


[GitHub] [incubator-uniffle] jerqi closed issue #127: [Improvement] Add timeout reconnection in DelegationRssShuffleManager

Posted by GitBox <gi...@apache.org>.
jerqi closed issue #127: [Improvement] Add timeout reconnection in DelegationRssShuffleManager
URL: https://github.com/apache/incubator-uniffle/issues/127


-- 
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@uniffle.apache.org

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


[GitHub] [incubator-uniffle] jerqi commented on issue #127: [Improvement] Add timeout reconnection in DelegationRssShuffleManager

Posted by GitBox <gi...@apache.org>.
jerqi commented on issue #127:
URL: https://github.com/apache/incubator-uniffle/issues/127#issuecomment-1206000568

   Similar pr: https://github.com/apache/incubator-uniffle/pull/115 , Could it solve your issue?


-- 
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@uniffle.apache.org

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


[GitHub] [incubator-uniffle] jerqi commented on issue #127: [Improvement] Add timeout reconnection in DelegationRssShuffleManager

Posted by GitBox <gi...@apache.org>.
jerqi commented on issue #127:
URL: https://github.com/apache/incubator-uniffle/issues/127#issuecomment-1207704851

   Solved by #139 


-- 
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@uniffle.apache.org

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


[GitHub] [incubator-uniffle] jerqi commented on issue #127: [Improvement] Add timeout reconnection in DelegationRssShuffleManager

Posted by GitBox <gi...@apache.org>.
jerqi commented on issue #127:
URL: https://github.com/apache/incubator-uniffle/issues/127#issuecomment-1206012541

   > Similar pr: #115 , Could it solve your issue?
   
   My fault. I misunderstand you. The improvement is acceptable for me. Do you want to raise a pr?


-- 
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@uniffle.apache.org

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