New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix S3 table function does not work for pre-signed URL #52310
Conversation
This is an automated comment for commit 0c3118c with description of existing statuses. It's updated for the latest CI running
|
@Avogar please review this pr. |
What is the idea? Did you test it? |
@nikitamikhaylov file_extension will return a string with a question mark, just extract the extension before the question mark. I have do the test, but aarch64 platform can not support to run integration test, there is a problem with the docker image. So I can not add integration test now. |
Actually,
But we can encode the Lines 59 to 64 in e133db7
So, |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix S3 table function does not work for pre-signed URL. close #50846
Documentation entry for user-facing changes