Skip to content
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-974548 Upgrade AWS SDK #971

Merged
merged 1 commit into from
Nov 28, 2023
Merged

SNOW-974548 Upgrade AWS SDK #971

merged 1 commit into from
Nov 28, 2023

Conversation

sfc-gh-pfus
Copy link
Collaborator

@sfc-gh-pfus sfc-gh-pfus commented Nov 20, 2023

Description

Upgraded AWS SDK library. Upgrade was needed as AWS client changed incompatibly.

Kudos for @kenshaw for suggesting the backward compatible solution!

Checklist

  • Code compiles correctly
  • Run make fmt to fix inconsistent formats
  • Run make lint to get lint errors and fix all of them
  • Created tests which fail without the change (if possible)
  • All tests passing
  • Extended the README / documentation, if necessary

Copy link

codecov bot commented Nov 20, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5c79db8) 82.78% compared to head (10c4c72) 82.80%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #971      +/-   ##
==========================================
+ Coverage   82.78%   82.80%   +0.01%     
==========================================
  Files          50       50              
  Lines        9633     9643      +10     
==========================================
+ Hits         7975     7985      +10     
  Misses       1658     1658              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sfc-gh-pfus sfc-gh-pfus marked this pull request as ready for review November 20, 2023 11:00
@sfc-gh-pfus sfc-gh-pfus requested a review from a team as a code owner November 20, 2023 11:00
serprex added a commit to PeerDB-io/peerdb that referenced this pull request Nov 20, 2023
Also: combine COPY/RUN commands in dockerfiles where applicable. Reduces size of images by reducing layers

Hold back gosnowflake/aws-sdk-go, gosnowflake changed default PUT to ovewrite=false in 1.7.0 & aws-sdk-go has breaking changes in aws-sdk-go 1.48
https://docs.snowflake.com/en/release-notes/clients-drivers/golang-2023

snowflakedb/gosnowflake#971
aws/aws-sdk-go-v2#2162
@sfc-gh-pfus sfc-gh-pfus force-pushed the SNOW-974548-aws-sdk branch 2 times, most recently from 00717bd to 3946751 Compare November 27, 2023 12:34
@sfc-gh-pfus sfc-gh-pfus merged commit f33dfc7 into master Nov 28, 2023
25 checks passed
@sfc-gh-pfus sfc-gh-pfus deleted the SNOW-974548-aws-sdk branch November 28, 2023 06:43
@github-actions github-actions bot locked and limited conversation to collaborators Nov 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants