blaginin opened a new pull request, #13576:
URL: https://github.com/apache/datafusion/pull/13576

   ## Which issue does this PR close?
   
   Related to https://github.com/apache/datafusion/issues/13456
   
   ## Rationale for this change
   
   Surprisingly, even the 
[simplest](https://datafusion.apache.org/user-guide/cli/datasources.html#s3) S3 
example doesn't work 😱 
   
   ```sql
   > CREATE EXTERNAL TABLE test
   STORED AS CSV
   OPTIONS(
       'aws.access_key_id' '************',
       'aws.secret_access_key' '**********',
       'aws.region' 'eu-west-1'
   )
   LOCATION 's3://blaginin/playground/titanic.csv';
   
   
   Object Store error: The operation lacked the necessary privileges to 
complete for path playground/titanic.csv: Client error with status 403 
Forbidden: No Body
   ```
   That's because s3 keys are case sensitive are case-sensitive, but now we 
always lowercase them.
   
   
   
   I see you had the same issue for HuggingFace so I really think the default 
behaviour should be NOT normalizing. 
   
   ## What changes are included in this PR?
   
   Change the default value
   
   ## Are these changes tested?
   
   🤔 
   
   ## Are there any user-facing changes?
   
   Changed the default value
   


-- 
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-unsubscr...@datafusion.apache.org

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


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

Reply via email to