You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/06/29 12:44:00 UTC

[GitHub] [spark] HyukjinKwon commented on pull request #28945: [MINOR][SQL] Fix spaces in JDBC connection providers

HyukjinKwon commented on pull request #28945:
URL: https://github.com/apache/spark/pull/28945#issuecomment-651094453


   It's actually not encouraged to make a PR for some trivial nits that are virtually same before/after in general given that we can just fix such nits later when we touch the codes for a fix together, and it uses the limited resources in Jenkins which abort the jobs globally when the number of jobs is too high for some reasons.
   
   At least this PR fixes all style nits under connection packages non-invasively so I guess it's fine but let's avoid next time.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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