davidm-db commented on code in PR #56329: URL: https://github.com/apache/spark/pull/56329#discussion_r3637389107
########## docs/control-flow/leave-stmt.md: ########## @@ -19,14 +19,14 @@ license: | limitations under the License. --- -Terminates the execution of an iteration of a looping statement and continues with the next iteration if the looping condition is met. +Exits the execution of a labeled statement, which may be a looping construct or a labeled compound block. Review Comment: I'd rephrase this a bit with a more strict wording - we either "exit the statement" or "terminate the execution". Also, I think that the we are using "statement" throughout the docs for all the constructs - so, "looping statement" and "compound statement" seem more consistent. Thus, I'd do something like: ```suggestion Exits a labeled looping statement or compound statement, transferring control to the statement that follows it. ``` or a simpler version ```suggestion Terminates the execution of a labeled looping or compound statement. ``` or whatever else you can think of that fits the best. ########## docs/control-flow/leave-stmt.md: ########## @@ -68,4 +68,3 @@ ITERATE label - [WHILE Statement](../control-flow/while-stmt.html) Review Comment: nit: while we are at it, let's also add a link to REPEAT statement here - other related statements have it. -- 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]
