You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/01/10 09:33:50 UTC

[GitHub] [spark] zero323 edited a comment on pull request #35150: [SPARK-37850][PYTHON][INFRA] Enable flake's E731 rule in PySpark

zero323 edited a comment on pull request #35150:
URL: https://github.com/apache/spark/pull/35150#issuecomment-1008688841


   Personally, I'd avoid this rule completely as, unlike broader Python community, we're kind of lambda friendly and, especially in deeply nested context, full defs seem like overkill :)
   
   That being said, LGTM, once test pass.


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org