AngersZhuuuu commented on pull request #30131: URL: https://github.com/apache/spark/pull/30131#issuecomment-716333959
> I agree with @Ngone51 that moving periodic gc from rate to delay is good, I am not sure about the rest - can you elaborate why the others are being made ? Any issues resolved/bugs fixed ? If not, limiting to gc change might be good. Not shown is change about RPC, since this change can avoid some unnecessary repeated RPC about `register and heartbeat`. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org