Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix EI-2347 #983

Merged
merged 1 commit into from
Jun 26, 2018
Merged

Fix EI-2347 #983

merged 1 commit into from
Jun 26, 2018

Conversation

sdkottegoda
Copy link
Contributor

Purpose

Fixes thread death when unsubscribing a subscription from UI.
Resolves: wso2/product-ei#2347

Approach

The underlying cause was a dead lock in the disruptor that occured when the DB recovery task runs. The DB recovery event(a disruptor event) detects conflicts of the connected node of local subscriptions and susbscriptions stored in the databse. Once a conflict is detected it disonnects the local subscription resulting in an event being published to the disruptor. This causes all disruptor handlers to hang thus MB
being barely usable. The resolution for this is decouple the local subscription being deleted from the detection of the conflict.

@sdkottegoda sdkottegoda self-assigned this Jun 22, 2018
Fixes thread death when unsubscribing a subscription from UI. The
underlying cause was a dead lock in the disruptor that occured when the
DB recovery task runs. The DB recovery event(a disruptor event) detects
conflicts of the connected node of local subscriptions and
susbscriptions stored in the databse. Once a conflict is detected it
disonnects the local subscription resulting in an event being published
to the disruptor. This causes all disruptor handlers to hang thus MB
being barely usable. The resolution for this is decouple the local
subscription being deleted from the detection of the conflict.

Resolves: wso2/product-ei#2347
@sdkottegoda sdkottegoda merged commit 2b7b4d3 into wso2:master Jun 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants