Skip to content

Add setting to allow different windows for stream and session - #8203

Closed
shinrich wants to merge 1 commit into
apache:masterfrom
shinrich:add-session-initial-window
Closed

Add setting to allow different windows for stream and session#8203
shinrich wants to merge 1 commit into
apache:masterfrom
shinrich:add-session-initial-window

Conversation

@shinrich

@shinrich shinrich commented Aug 3, 2021

Copy link
Copy Markdown
Member

Issue #8199 outlines the issue. This PR adds a new setting to allow for setting the session window separately from the stream window.

This closes #8199

@shinrich shinrich added the HTTP/2 label Aug 3, 2021
@shinrich shinrich added this to the 10.0.0 milestone Aug 3, 2021
@shinrich shinrich self-assigned this Aug 3, 2021
@shinrich

shinrich commented Aug 4, 2021

Copy link
Copy Markdown
Member Author

[approve ci clang-analyzer]

@bryancall

Copy link
Copy Markdown
Contributor

@masaori335 or @maskit can you please take a look at this PR

@maskit maskit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As I commented on the issue, I don't think this is a good solution. -0 from me.

@bryancall

Copy link
Copy Markdown
Contributor

[approve ci]

1 similar comment
@bryancall

Copy link
Copy Markdown
Contributor

[approve ci]

@github-actions

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

@github-actions github-actions Bot added the Stale label Jan 13, 2022
@github-actions github-actions Bot closed this Jan 20, 2022
@zwoop zwoop removed this from the 10.0.0 milestone Jan 25, 2022
@maskit maskit mentioned this pull request Aug 9, 2022
@bneradt

bneradt commented Sep 10, 2022

Copy link
Copy Markdown
Contributor

Let's re-open this and pull it in separately from the new HTTP/2 to origin PR (#8963).

@bneradt

bneradt commented Sep 10, 2022

Copy link
Copy Markdown
Contributor

Since this has conflicts, I opened #9085 to take over from this PR.

@bneradt bneradt closed this Sep 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should have separate values for stream and session windows in HTTP/2

5 participants