dcoliversun commented on pull request #35758: URL: https://github.com/apache/spark/pull/35758#issuecomment-1062460450
> Hm, I don't think this is worthwhile. Either way with/without braces is fine to me honestly. @HyukjinKwon Hi. Code like the following is syntactically correct; but it is bug-prone ```java if (condition) statements; ``` The problem is sooner or later you or someone else is going to need add a line to that if statement. ```java if (condition) statements; statements; ``` The program has suddenly become buggy. Not everyone does when the buggy lines aren’t called out and pointed to as they are here. Even written like this, not everyone will see the bug immediately. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
