net/http: bundled http2 uses priority-based scheduler instead of round robin scheduler

[WeidiDeng]

What version of Go are you using (go version)?

$ go version
1.21.4

Does this issue reproduce with the latest release?

Yes

What did you expect to see?

Bundled http2 uses round robin scheduler.

What did you see instead?

By default, bundled http2 support should use round robin scheduler according to this comment and commit. Howerver, it will still use the old priority scheduler because this field is set, which prevented the default behavior from happening.

[gopherbot]

Change https://go.dev/cl/543178 mentions this issue: net/http: bundled http2 uses the new round-robin scheduler

[bcmills]

(attn @neild)

[linxiulei]

Gentle ping @WeidiDeng. Your CL https://go.dev/cl/543178 is ready to submit. Would you like to submit it?

[bcmills]

I think at this point in the release cycle, changing the http2 scheduler would need a freeze exception from @golang/release — RC1 is out for testing and does not include the change, which seems like it may be significant for some users.

[linxiulei]

Duplicate of #67706

I think it can be closed now.

[WeidiDeng]

@linxiulei I was the first to report the problem and I made a pr that addresses this problem, but it never got merged. I would call #67706 a duplicate of this instead of the other way round.

It is fixed, so whatever 🤷 .