Skip to content

Commit 50303a0

Browse files
committed
Update "Add explicit flush support to Log4j AsyncAppender"
1 parent a2f4d34 commit 50303a0

2 files changed

Lines changed: 1 addition & 1 deletion

File tree

paper-server/patches/features_unapplied/0032-Add-explicit-flush-support-to-Log4j-AsyncAppender.patch renamed to paper-server/patches/features/0024-Add-explicit-flush-support-to-Log4j-AsyncAppender.patch

File renamed without changes.

paper-server/src/main/java/io/papermc/paper/plugin/manager/PaperPluginInstanceManager.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -343,7 +343,7 @@ private void flushAsyncAppenders() {
343343

344344
for (final Appender appender : context.getConfiguration().getAppenders().values()) {
345345
if (appender instanceof AsyncAppender asyncAppender) {
346-
final boolean flushed = true; // TODO - snapshot - feature patches - asyncAppender.flush(100, TimeUnit.MILLISECONDS);
346+
final boolean flushed = asyncAppender.flush(100, TimeUnit.MILLISECONDS);
347347
if (!flushed) {
348348
this.server.getLogger().log(Level.WARNING, "Failed to flush log messages before plugin unload.");
349349
}

0 commit comments

Comments
 (0)