You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/10/12 20:16:42 UTC

[GitHub] [arrow-datafusion] andygrove commented on pull request #3385: Improve doc on lowercase treatment of columns on SQL

andygrove commented on PR #3385:
URL: https://github.com/apache/arrow-datafusion/pull/3385#issuecomment-1276685509

   > What do you think about changing the introductory example in the user guide @andygrove ? Given it is the first thing some people see I think there is a potential reason to keep it as simple as possible (and thus maybe it would be nice to leave it uncapitalized, and add add a link about the important subtlety of capitalization to somewhere later?
   > 
   > If you agree I will merge this PR and then make a follow on PR that tries to retain the simple initial example as well as the capitalized example
   
   That sounds great. Thanks @alamb 


-- 
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: github-unsubscribe@arrow.apache.org

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