Skip to content

Commit 0f31c38

Browse files
committed
Wait 2 secondes before closing MockFluentd. Wait 5 seconds before closing MockFluentd if some worker threads are still running
1 parent fe67a1c commit 0f31c38

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

src/test/java/org/fluentd/logger/TestFluentLogger.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,7 @@ public void process(MessagePack msgpack, Socket socket) throws IOException {
8787

8888
// close loggers
8989
logger.close();
90+
Thread.sleep(2000);
9091

9192
// close mock fluentd
9293
fluentd.close();
@@ -169,6 +170,7 @@ public void process(MessagePack msgpack, Socket socket) throws IOException {
169170

170171
// close loggers
171172
FluentLogger.closeAll();
173+
Thread.sleep(2000);
172174

173175
// close mock fluentd
174176
fluentd.close();

src/test/java/org/fluentd/logger/util/MockFluentd.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,9 +146,12 @@ public void close() throws IOException {
146146
try {
147147
// We need to wait until all log writing threads are finished.
148148
int numTrial = 0;
149-
while(numTrial < 3 && !service.awaitTermination(1, TimeUnit.SECONDS)) {
149+
final int maxTrial = 5;
150+
while(numTrial < 5 && !service.awaitTermination(1, TimeUnit.SECONDS)) {
150151
numTrial++;
151152
}
153+
if(numTrial >= maxTrial)
154+
_logger.error("Timed out");
152155
}
153156
catch(InterruptedException e) {
154157
_logger.error("interrupted", e);

0 commit comments

Comments
 (0)