Skip to content

Commit c5cdcac

Browse files
author
Darren Stack
committed
fix: Use sagemaker session's s3 resource for downloading folders
1 parent a5c6229 commit c5cdcac

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/sagemaker/utils.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -384,8 +384,7 @@ def download_folder(bucket_name, prefix, target, sagemaker_session):
384384
sagemaker_session (sagemaker.session.Session): a sagemaker session to
385385
interact with S3.
386386
"""
387-
boto_session = sagemaker_session.boto_session
388-
s3 = boto_session.resource("s3", region_name=boto_session.region_name)
387+
s3 = sagemaker_session.s3_resource
389388

390389
prefix = prefix.lstrip("/")
391390

@@ -657,9 +656,8 @@ def download_file(bucket_name, path, target, sagemaker_session):
657656
interact with S3.
658657
"""
659658
path = path.lstrip("/")
660-
boto_session = sagemaker_session.boto_session
661659

662-
s3 = boto_session.resource("s3", region_name=sagemaker_session.boto_region_name)
660+
s3 = sagemaker_session.s3_resource
663661
bucket = s3.Bucket(bucket_name)
664662
bucket.download_file(path, target)
665663

0 commit comments

Comments
 (0)