cloud-fan commented on pull request #33287:
URL: https://github.com/apache/spark/pull/33287#issuecomment-878335670


   `\b` means backspace, which is a control character that moves the cursor one 
character back in the console but doesn't delete it. I don't think we should 
trim it as whitespace.
   
   I think this is a doc issue. The intention is to trim white spaces, but the 
condition (<= ASCII 32) was wrong and 
https://github.com/apache/spark/pull/29375 fixed it. Shall we fix the 
doc/comment instead?


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

Reply via email to