[GitHub] [spark] dongjoon-hyun edited a comment on pull request #29121: [SPARK-32319][PYSPARK] Remove unused imports

2020-07-15 Thread GitBox
dongjoon-hyun edited a comment on pull request #29121: URL: https://github.com/apache/spark/pull/29121#issuecomment-658958309 It would be great if you mention `suppressing` in the PR title and PR description. Otherwise, the PR title is misleading. -

[GitHub] [spark] dongjoon-hyun edited a comment on pull request #29121: [SPARK-32319][PYSPARK] Remove unused imports

2020-07-15 Thread GitBox
dongjoon-hyun edited a comment on pull request #29121: URL: https://github.com/apache/spark/pull/29121#issuecomment-658818917 This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [spark] dongjoon-hyun edited a comment on pull request #29121: [SPARK-32319][PYSPARK] Remove unused imports

2020-07-15 Thread GitBox
dongjoon-hyun edited a comment on pull request #29121: URL: https://github.com/apache/spark/pull/29121#issuecomment-658818917 I also agree with @HyukjinKwon . Without the rule enforcement, `unused import` always happens again because we review and merge PR in a parallel way Also,

[GitHub] [spark] dongjoon-hyun edited a comment on pull request #29121: [SPARK-32319][PYSPARK] Remove unused imports

2020-07-15 Thread GitBox
dongjoon-hyun edited a comment on pull request #29121: URL: https://github.com/apache/spark/pull/29121#issuecomment-658818917 I also agree with @HyukjinKwon . Without the rule enforcement, `unused import` always happens again because we review and merge PR in a parallel way Also,

[GitHub] [spark] dongjoon-hyun edited a comment on pull request #29121: [SPARK-32319][PYSPARK] Remove unused imports

2020-07-15 Thread GitBox
dongjoon-hyun edited a comment on pull request #29121: URL: https://github.com/apache/spark/pull/29121#issuecomment-658818917 I also agree with @HyukjinKwon . Without the rule enforcement, `unused import` always happens again because we review and merge PR in a parallel way. --