[issue2073] asynchat push always sends 512 bytes (ignoring ac_out_buffer_size)

[Mike Coleman]

Mike Coleman <mkc at users.sourceforge.net> added the comment:

Just to confirm, the real problem here was that tiny packets were being
sent out by default, and the obvious fix (altering ac_out_buffer_size)
didn't work.

Looking at the code, it appears that the change by Josiah Carlson
(#64062) causes ac_out_buffer_size (which currently defaults to 4096) to
be obeyed.

Should that 4096 be larger?  MTU for jumbo packets is ~9000, I think,
plus there's less system call overhead for larger values.  (I use 62000
for my app, but I haven't studied alternatives lately.)

Also, there are still two constants '512' in asyncore/asynchat. 
Shouldn't these be changed to match the 4096 default?

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue2073>
_______________________________________