Qortal/altcoinj
3
0
Fork 0
mirror of https://github.com/Qortal/altcoinj.git synced 2025-02-12 10:15:52 +00:00
Code Issues Projects Releases Wiki Activity

Threading: prefer OOM to deadlock when the user thread is saturated, but keep the warning.

Browse Source
This commit is contained in:
Mike Hearn 2014-01-15 17:56:10 +01:00
parent a9a7dd9e06
commit 80d4840199
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
core/src/main/java/com/google/bitcoin/utils/Threading.java
View File

@ -84,10 +84,10 @@ public class Threading {
private static final Logger log = LoggerFactory.getLogger(UserThread.class);
private LinkedBlockingQueue<Runnable> tasks;
public UserThread(int tasksBound) {
public UserThread() {
super("bitcoinj user thread");
setDaemon(true);
tasks = new LinkedBlockingQueue<Runnable>(tasksBound);
tasks = new LinkedBlockingQueue<Runnable>();
start();
}
@ -108,13 +108,12 @@ public class Threading {
@Override
public void execute(Runnable command) {
// Will block if the event thread is saturated.
if (!tasks.offer(command)) {
log.warn("User thread saturated, check for deadlocked or slow event handlers. Sample tasks:");
if (tasks.size() > 100) {
log.warn("User thread saturated, memory exhaustion may occur.");
log.warn("Check for deadlocked or slow event handlers. Sample tasks:");
for (Object task : tasks.toArray()) log.warn(task.toString());
// Try again and wait this time.
Uninterruptibles.putUninterruptibly(tasks, command);
}
Uninterruptibles.putUninterruptibly(tasks, command);
}
}
@ -124,7 +123,7 @@ public class Threading {
// from that point onwards.
throwOnLockCycles();
USER_THREAD = new UserThread(100); // 100 pending event listeners to avoid memory blowup.
USER_THREAD = new UserThread();
SAME_THREAD = new Executor() {
@Override
public void execute(@Nonnull Runnable runnable) {