161sh/CatgirlIntelligenceAgency
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix bug in ProcessingIterator where it would run the tasks in only one single thread instead of using the pool

Browse Source
This commit is contained in:
Viktor Lofgren 2023-12-29 15:10:32 +01:00
parent a1f3ccdd6d
commit ba8a75c84b
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
code/libraries/blocking-thread-pool/src/main/java/nu/marginalia/util/ProcessingIterator.java
View File

@ -38,7 +38,7 @@ public class ProcessingIterator<T> implements Iterator<T> {
private void executeJob(ProcessingJob<T> job) {
try {
job.run(j -> executorService.submit(() -> executeTask(j)));
job.run(this::executeTask);
} catch (Exception e) {
logger.warn("Exception while processing", e);
} finally {
@ -53,13 +53,15 @@ public class ProcessingIterator<T> implements Iterator<T> {
return;
}
try {
queue.put(task.get());
} catch (Exception e) {
logger.warn("Exception while processing", e);
} finally {
sem.release();
}
executorService.submit(() -> {
try {
queue.put(task.get());
} catch (Exception e) {
logger.warn("Exception while processing", e);
} finally {
sem.release();
}
});
}
/** Returns true if there are more documents to be processed.