There's some cleanup to be done in terms of changing the queue to work on TaskWrappers, removing logging. Otherwise this should be good to go.
Siddharth Seth
added a comment - prasanth_j - could you please take a quick look ?
There's some cleanup to be done in terms of changing the queue to work on TaskWrappers, removing logging. Otherwise this should be good to go.
Updated patch. Fixes a potential deadlock in the previous one, and another condition where queued tasks may be put into the preemption queue.
Locking could potentially be reduced - but I'm going to create a separate jira for that.
Siddharth Seth
added a comment - Updated patch. Fixes a potential deadlock in the previous one, and another condition where queued tasks may be put into the preemption queue.
Locking could potentially be reduced - but I'm going to create a separate jira for that.
prasanth_j - could you please take a quick look ?
There's some cleanup to be done in terms of changing the queue to work on TaskWrappers, removing logging. Otherwise this should be good to go.