Skip to content

Commit

Permalink
Remove pool context manager
Browse files Browse the repository at this point in the history
  • Loading branch information
obulat committed Mar 28, 2023
1 parent 6fbb196 commit 2b52f30
Showing 1 changed file with 13 additions and 10 deletions.
23 changes: 13 additions & 10 deletions ingestion_server/ingestion_server/cleanup.py
Original file line number Diff line number Diff line change
Expand Up @@ -351,16 +351,19 @@ def clean_image_data(table):
cleanable_fields_for_table,
)
)
with multiprocessing.Pool(processes=num_workers) as pool:
log.info(f"Starting {len(jobs)} cleaning jobs")

for result in pool.starmap(_clean_data_worker, jobs):
batch_cleaned_counts = save_cleaned_data(result)
log.info(f"Batch cleaned counts: {batch_cleaned_counts}")
log.info(f"Multiprocessing batch finished, result: {result}")
for field in batch_cleaned_counts:
cleaned_counts_by_field[field] += batch_cleaned_counts[field]
log.info("Finished cleaning jobs")
pool = multiprocessing.Pool(processes=num_workers)
log.info(f"Starting {len(jobs)} cleaning jobs")

results = pool.starmap(_clean_data_worker, jobs)
log.info(f"Multiprocessing pool finished, results: {results}")
for result in results:
batch_cleaned_counts = save_cleaned_data(result)
log.info(f"Batch cleaned counts: {batch_cleaned_counts}")
log.info(f"Multiprocessing batch finished, result: {result}")
for field in batch_cleaned_counts:
cleaned_counts_by_field[field] += batch_cleaned_counts[field]
pool.close()
log.info("Finished cleaning jobs")
log.info("Finished multiprocessing pool")
num_cleaned += len(batch)
batch_end_time = time.time()
Expand Down

0 comments on commit 2b52f30

Please sign in to comment.