Skip to content

Commit 8fe7800

Browse files
author
Nilesh PS
committed
fix: Remove confusing log line emitted during feature group ingestion
1 parent 7faf9ce commit 8fe7800

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/sagemaker/feature_store/feature_group.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -365,12 +365,10 @@ def _run_multi_threaded(
365365
failed_indices = list()
366366
for future in as_completed(futures, timeout=timeout):
367367
start, end = futures[future]
368-
result = future.result()
369-
if result:
370-
logger.error("Failed to ingest row %d to %d", start, end)
371-
else:
368+
failed_rows = future.result()
369+
if not failed_rows:
372370
logger.info("Successfully ingested row %d to %d", start, end)
373-
failed_indices += result
371+
failed_indices += failed_rows
374372

375373
executor.shutdown(wait=False)
376374

0 commit comments

Comments
 (0)