Skip to content

Commit 58c145c

Browse files
committed
don't have to check urgent mode when starting saving task
1 parent a347174 commit 58c145c

File tree

1 file changed

+9
-18
lines changed

1 file changed

+9
-18
lines changed

agent/storage/src/main/java/me/hsgamer/topper/agent/storage/StorageAgent.java

Lines changed: 9 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
import me.hsgamer.topper.storage.core.DataStorage;
88

99
import java.util.*;
10-
import java.util.concurrent.CompletableFuture;
1110
import java.util.concurrent.ConcurrentLinkedQueue;
1211
import java.util.concurrent.atomic.AtomicBoolean;
1312
import java.util.concurrent.atomic.AtomicReference;
@@ -66,24 +65,16 @@ private void save(boolean urgent) {
6665
})
6766
.collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
6867

69-
Runnable saveTask = () -> {
70-
try {
71-
storage.save(finalMap);
72-
if (!removeKeys.isEmpty()) {
73-
storage.remove(removeKeys);
74-
}
75-
savingMap.set(null);
76-
} catch (Throwable t) {
77-
logger.log(Level.SEVERE, "Failed to save entries for " + holder.getName(), t);
78-
} finally {
79-
saving.set(false);
68+
try {
69+
storage.save(finalMap);
70+
if (!removeKeys.isEmpty()) {
71+
storage.remove(removeKeys);
8072
}
81-
};
82-
83-
if (urgent) {
84-
saveTask.run();
85-
} else {
86-
CompletableFuture.runAsync(saveTask);
73+
savingMap.set(null);
74+
} catch (Throwable t) {
75+
logger.log(Level.SEVERE, "Failed to save entries for " + holder.getName(), t);
76+
} finally {
77+
saving.set(false);
8778
}
8879
}
8980

0 commit comments

Comments
 (0)