jerqi commented on issue #89:
URL: 
https://github.com/apache/incubator-uniffle/issues/89#issuecomment-1196332327

   The idea is acceptable. But there are some questions
   1. Why do we need HealthCheck? HealthCheck is an experimental feature now, 
maybe it's not stable enough. That means that you need do some extra work to 
improve the feature.
   2. Should we add some extra metrics to measure the effect of strategy?  We 
usually use the memory of server to measure current strategy, we hope every 
server use similar quantity of memory.


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

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

Reply via email to