-
Notifications
You must be signed in to change notification settings - Fork 28
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
SNOW-1524269: support put/get for GCP #738
SNOW-1524269: support put/get for GCP #738
Conversation
a45b4df
to
accc73b
Compare
5f1348c
to
4ace536
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we need to merge this feature to master 2.0? there are no incompatible changes with the current 1.x
Noted down in the ticket discussed about that with @sfc-gh-dstempniak before. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's remove presigned url related code
teamwork issue 1035
part 1 for GCP support, will have separated PR to support PUT/GET natively.
Draft for now as temporary changes included to fake the client to be ODBC to pass test. Without that we would get error of "gcs downscoped token not supported" from server.
We would need server change to enable PUT/GET on GCS for libsnowflakeclient (maybe for PHP as well).