Investigate why sockjs-protocol tests time out with xitrum-sockjs-test + Netty 4.0.16 and 4.0.17 #309

Closed
ngocdaothanh opened this Issue Feb 27, 2014 · 3 comments

Comments

Projects
None yet
2 participants
@ngocdaothanh
Member

ngocdaothanh commented Feb 27, 2014

http://netty.io/news/2014/02/25/4-0-17-Final.html

xitrum-sockjs-test is fast as normal with Netty 4.0.15 and below.

@ngocdaothanh ngocdaothanh added this to the Xitrum 3.2 milestone Feb 27, 2014

@ngocdaothanh ngocdaothanh self-assigned this Feb 27, 2014

@ngocdaothanh

This comment has been minimized.

Show comment Hide comment
@ngocdaothanh

ngocdaothanh Feb 27, 2014

Member

It's maybe because with the new Netty versions, NoPipelining.pauseReading and NoPipelining.resumeReading do not work fast as before.

Asked on Stack Overflow:
http://stackoverflow.com/questions/22060454/how-to-pause-and-resume-reading-with-netty-4

Member

ngocdaothanh commented Feb 27, 2014

It's maybe because with the new Netty versions, NoPipelining.pauseReading and NoPipelining.resumeReading do not work fast as before.

Asked on Stack Overflow:
http://stackoverflow.com/questions/22060454/how-to-pause-and-resume-reading-with-netty-4

@ngocdaothanh

This comment has been minimized.

Show comment Hide comment
@ngocdaothanh

ngocdaothanh Apr 10, 2014

Member

See also netty/netty#2375
Fix bug in respecting ChannelConfig.setAutoRead(false)

Member

ngocdaothanh commented Apr 10, 2014

See also netty/netty#2375
Fix bug in respecting ChannelConfig.setAutoRead(false)

@ngocdaothanh ngocdaothanh modified the milestones: Xitrum 3.7, Xitrum 3.6 Apr 10, 2014

@ngocdaothanh

This comment has been minimized.

Show comment Hide comment
@ngocdaothanh

ngocdaothanh Apr 11, 2014

Member

Explained and fixed at #347.

Member

ngocdaothanh commented Apr 11, 2014

Explained and fixed at #347.

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