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, we cannot `Remove unused imports` in this PR. This PR simply suppresses warnings in some cases instead of `remove unused imports`. ```python from pyspark.sql import SQLContext, HiveContext, Row # noqa: F401 ``` Sorry, @Fokko . Given the above two factors, I'm -1 for this AS-IS PR. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
