[GitHub] [spark] AngersZhuuuu commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter

2020-03-24 Thread GitBox
AngersZh commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter URL: https://github.com/apache/spark/pull/28011#issuecomment-603666860 > Yes, because it costs Jenkins, review and merge. Also, it's difficult to maintain such PRs e

[GitHub] [spark] AngersZhuuuu commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter

2020-03-24 Thread GitBox
AngersZh commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter URL: https://github.com/apache/spark/pull/28011#issuecomment-603663192 > No~ @AngersZh. I meant to make one minor PR to fix many typos instead of making one PR t

[GitHub] [spark] AngersZhuuuu commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter

2020-03-24 Thread GitBox
AngersZh commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter URL: https://github.com/apache/spark/pull/28011#issuecomment-603657534 cc @HyukjinKwon Split to two pr, thanks --