Skip to content

Track scheduled events to (read|write)_vio.cont from Http2Stream#5934

Merged
masaori335 merged 1 commit intoapache:masterfrom
masaori335:4921
Sep 24, 2019
Merged

Track scheduled events to (read|write)_vio.cont from Http2Stream#5934
masaori335 merged 1 commit intoapache:masterfrom
masaori335:4921

Conversation

@masaori335
Copy link
Copy Markdown
Contributor

Possible fix for #4921.

There is a path of Http2Stream schedules VC_EVENT_WRITE_COMPLETE(103) event to HttpTunnel (write_vio.cont). The event should be tracked and canceled when HttpSM and Http2Stream are closed.

@masaori335 masaori335 added this to the 10.0.0 milestone Sep 13, 2019
@masaori335 masaori335 self-assigned this Sep 13, 2019
Copy link
Copy Markdown
Member

@shinrich shinrich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@masaori335 masaori335 merged commit 9bd2fd6 into apache:master Sep 24, 2019
@zwoop zwoop modified the milestones: 10.0.0, 9.0.0 Oct 1, 2019
@zwoop
Copy link
Copy Markdown
Contributor

zwoop commented Oct 1, 2019

Cherry-picked to v9.0.x branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants