Skip to content

Commit ee77880

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

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/sagemaker/feature_store/feature_group.py

+3-5
Original file line numberDiff line numberDiff line change
@@ -427,12 +427,10 @@ def _run_multi_threaded(
427427
failed_indices = list()
428428
for future in as_completed(futures, timeout=timeout):
429429
start, end = futures[future]
430-
result = future.result()
431-
if result:
432-
logger.error("Failed to ingest row %d to %d", start, end)
433-
else:
430+
failed_rows = future.result()
431+
if not failed_rows:
434432
logger.info("Successfully ingested row %d to %d", start, end)
435-
failed_indices += result
433+
failed_indices += failed_rows
436434

437435
executor.shutdown(wait=False)
438436

0 commit comments

Comments
 (0)