Skip to content

Commit d3aa98c

Browse files
psnileshNilesh PS
and
Nilesh PS
authored
fix: Remove confusing log line emitted during feature group ingestion (#3597)
Co-authored-by: Nilesh PS <[email protected]>
1 parent bde191f commit d3aa98c

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
@@ -421,12 +421,10 @@ def _run_multi_threaded(
421421
failed_indices = list()
422422
for future in as_completed(futures, timeout=timeout):
423423
start, end = futures[future]
424-
result = future.result()
425-
if result:
426-
logger.error("Failed to ingest row %d to %d", start, end)
427-
else:
424+
failed_rows = future.result()
425+
if not failed_rows:
428426
logger.info("Successfully ingested row %d to %d", start, end)
429-
failed_indices += result
427+
failed_indices += failed_rows
430428

431429
executor.shutdown(wait=False)
432430

0 commit comments

Comments
 (0)