Skip to content

Commit e0ed8b7

Browse files
committed
Fix unit test failing with ConcurrentModificationException; during close
all, take a copy of the list of loggers for iteration
1 parent 60187cf commit e0ed8b7

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/main/java/org/fluentd/logger/FluentLoggerFactory.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919

2020
import java.lang.reflect.Constructor;
2121
import java.lang.reflect.InvocationTargetException;
22+
import java.util.ArrayList;
2223
import java.util.Iterator;
2324
import java.util.Map;
2425
import java.util.Map.Entry;
@@ -112,7 +113,7 @@ Map<FluentLogger, String> getLoggers() {
112113
}
113114

114115
public synchronized void closeAll() {
115-
for (FluentLogger logger : loggers.keySet()) {
116+
for (FluentLogger logger : new ArrayList<FluentLogger>(loggers.keySet())) {
116117
logger.close();
117118
}
118119
loggers.clear();

0 commit comments

Comments
 (0)