Github user srowen commented on the pull request:

    https://github.com/apache/spark/pull/8795#issuecomment-141019047
  
    This is a good fix, but this is another instance where I suspect the same 
issue exists in several markup files, like `configuration.html`. It's worth a 
JIRA since I think catching and fixing all of these is one non-trivial logical 
change.
    
    If you can, avoid whitespace changes like stripping or adding space at the 
end of lines. It just adds to the diff and makes for a tiny extra chance of 
merge conflicts.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to