Skip to content

Commit 5938849

Browse files
author
Kusumlata Bhatt
committed
fix: Extracted profile_name directly from sagemaker.Session if None
1 parent b731021 commit 5938849

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

src/sagemaker/feature_store/feature_group.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -804,6 +804,9 @@ def ingest(
804804
if max_workers <= 0:
805805
raise RuntimeError("max_workers must be greater than 0.")
806806

807+
if profile_name is None:
808+
profile_name = self.sagemaker_session.boto_session.profile_name
809+
807810
manager = IngestionManagerPandas(
808811
feature_group_name=self.name,
809812
sagemaker_session=self.sagemaker_session,

tests/unit/sagemaker/feature_store/test_feature_group.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ def test_ingest(ingestion_manager_init, sagemaker_session_mock, fs_runtime_clien
311311
sagemaker_fs_runtime_client_config=fs_runtime_client_config_mock,
312312
max_workers=10,
313313
max_processes=1,
314-
profile_name=None,
314+
profile_name=sagemaker_session_mock.boto_session.profile_name,
315315
)
316316
mock_ingestion_manager_instance.run.assert_called_once_with(
317317
data_frame=df, wait=True, timeout=None
@@ -337,7 +337,7 @@ def test_ingest_default(ingestion_manager_init, sagemaker_session_mock):
337337
sagemaker_fs_runtime_client_config=fs_runtime_client_config_mock,
338338
max_workers=1,
339339
max_processes=1,
340-
profile_name=None,
340+
profile_name=sagemaker_session_mock.boto_session.profile_name,
341341
)
342342
mock_ingestion_manager_instance.run.assert_called_once_with(
343343
data_frame=df, wait=True, timeout=None

0 commit comments

Comments
 (0)