[GitHub] [spark] panbingkun commented on pull request #37148: [MINOR] Remove redundant return
panbingkun commented on PR #37148: URL: https://github.com/apache/spark/pull/37148#issuecomment-1180445408 > Merged to master. I am very sorrry, I miss some scala code with the same problem. @HyukjinKwon Followup 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. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] [spark] panbingkun commented on pull request #37148: [MINOR] Remove redundant return
panbingkun commented on PR #37148: URL: https://github.com/apache/spark/pull/37148#issuecomment-1179888952 > Are they all in the codebase? Yes, I inspect all scala code in spark by > Tool: "/Applications/IntelliJ IDEA CE.app/Contents/bin/inspect.sh" > Rule: Syntactic simplification - redundant return -- 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org