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