Skip to content

[FIX] Outgoing integrations were stopping the oplog tailing sometimes#11333

Merged
rodrigok merged 1 commit intodevelopfrom
hotfix/outgoing-integration-freeze
Jul 4, 2018
Merged

[FIX] Outgoing integrations were stopping the oplog tailing sometimes#11333
rodrigok merged 1 commit intodevelopfrom
hotfix/outgoing-integration-freeze

Conversation

@rodrigok
Copy link
Copy Markdown
Member

@rodrigok rodrigok commented Jul 4, 2018

When the integrations has a script that access some kind of async (made sync by Fibers) method the VM timeout would be fired after the timeout time and would generate some zoombie fibers cousing the oplog tailing (that uses Fibers) to stop working util a cursor timeout.

@rodrigok rodrigok added this to the 0.66.1 milestone Jul 4, 2018
@rodrigok rodrigok requested a review from sampaiodiego July 4, 2018 12:07
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11333 July 4, 2018 12:07 Inactive
When the integrations has a script that access some kind of async (made sync by Fibers) method the VM timeout would be fired after the timeout time and would generate some zoombie fibers cousing the oplog tailing (that uses Fibers) to stop working util a cursor timeout.
@RocketChat RocketChat deleted a comment Jul 4, 2018
@RocketChat RocketChat deleted a comment Jul 4, 2018
@rodrigok rodrigok merged commit 851c93d into develop Jul 4, 2018
@rodrigok rodrigok deleted the hotfix/outgoing-integration-freeze branch July 4, 2018 13:23
rodrigok added a commit that referenced this pull request Jul 4, 2018
…#11333)

When the integrations has a script that access some kind of async (made sync by Fibers) method the VM timeout would be fired after the timeout time and would generate some zoombie fibers cousing the oplog tailing (that uses Fibers) to stop working util a cursor timeout.
@rodrigok rodrigok mentioned this pull request Jul 4, 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.

3 participants