Junfan Zhang
Junfan Zhang
This optimization also could be applied on Spark. One concern is that whether it will put more pressure on shuffle-server, like IO/cpu ?
The similar mechanism is supported by other systems, like Flink: https://nightlies.apache.org/flink/flink-docs-release-1.13/docs/deployment/metric_reporters/ I think this is a good improvement of observability
Let's reopen this?
cc @zhengchenyu could you help review this ?
@qijiale76 Do you want to push this forward?
Could you help fix this? @lifeSo
> Hi @zuston , could you please take a look whether it is met expectation? Just take a look. Overall LGTM.
And I think the PR title is not proper for this change, which don't reflect the feature of overwriting the existing client conf.