]> source.dussan.org Git - vaadin-framework.git/commit
Avoid deadlock checking in runPendingAccessTasks() (#11897)
authorLeif Åstrand <leif@vaadin.com>
Tue, 28 May 2013 13:52:10 +0000 (16:52 +0300)
committerVaadin Code Review <review@vaadin.com>
Tue, 28 May 2013 15:56:47 +0000 (15:56 +0000)
commitf8319ebede962a9c89e192ab0496ba67fe2e4575
tree8818a5909cd3cb2df2ad2a0a334ef3383bb0af06
parentb2137f5d4bd00e11142f31f542b0f8d6ea98f518
Avoid deadlock checking in runPendingAccessTasks() (#11897)

The deadlock check in accessSynchronously() was triggered in cases where
one session was locked and tryLock() in access() did manage to lock
another session and thus attempted to run the runnable right away.

runPendingAccessTasks() needed accessSynchronously() just for setting up
the current session instance - the session was already locked and
cleaning up the CurrentInstance values afterwards was not needed. Just
setting the current session without accessSynchronously() gets rid of
the inappropriate deadlock detection as well as some overhead with
managing locks and CurrentInstance values.

Change-Id: Ib956d68884d6703dce2633a455c4857bb06651af
server/src/com/vaadin/server/VaadinSession.java