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/12/02 03:30:03 UTC

[GitHub] [incubator-uniffle] zuston opened a new issue, #384: [Improvement] Make coordinator class more organized

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

   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   
   
   ### Search before asking
   
   - [X] I have searched in the [issues](https://github.com/apache/incubator-uniffle/issues?q=is%3Aissue) and found no similar issues.
   
   
   ### What would you like to be improved?
   
   Currently, the class in coordinator package are all in single folder. We should make coordinator class more organized, and split them into different folders by their correlation.
   
   ### How should we improve?
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit 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.apache.org

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


[GitHub] [incubator-uniffle] zuston commented on issue #384: [Improvement] Make coordinator class more organized

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

   Do you want to pick up this issue? @smallzhongfeng I could assign to you 


-- 
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] zuston closed issue #384: [Improvement] Make coordinator class more organized

Posted by GitBox <gi...@apache.org>.
zuston closed issue #384: [Improvement] Make coordinator class more organized
URL: https://github.com/apache/incubator-uniffle/issues/384


-- 
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] zuston commented on issue #384: [Improvement] Make coordinator class more organized

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

   Oh. I encounter the github bug, which creates multiple duplicate issues.


-- 
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 #384: [Improvement] Make coordinator class more organized

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

   cc @smallzhongfeng 


-- 
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 #384: [Improvement] Make coordinator class more organized

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

   Sounds great. I think it is reasonable. At present, there are many classes of coordinator, including checkers, server selection strategy, etc. we can make a unified classification.


-- 
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 #384: [Improvement] Make coordinator class more organized

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

   Ok, let me come and please assign it to me. @zuston 


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