[jitsi-dev] PseudoTCP performance improvement


#1

Hi,
I've created two small performance improvements in PseudoTCP (see attached patch). This patch does the following:

* Replaces clockThread in PseudoTcpSocketImpl with a thread-pool. That way concurrent connections will use fewer threads.

* Checks logger.isLoggable(Level.FINER) when doing delayed ack to avoid unnecessary string concatenation.

Feel free to use it however you like!

Best Regards
Carl Hasselskog
Co-founder and CEO
Degoo Backup AB
carl@degoo.com<mailto:carl@degoo.com>
Phone: +46 73 070 1821
http://degoo.com<http://degoo.com/>
http://twitter.com/#!/DegooBackup

PseudoTCP-ThreadPool.patch (6.22 KB)


#2

Hi Carl,

Hi,

I’ve created two small performance improvements in PseudoTCP (see attached
patch). This patch does the following:

· Replaces clockThread in PseudoTcpSocketImpl with a thread-pool.
That way concurrent connections will use fewer threads.

· Checks logger.isLoggable(Level.FINER) when doing delayed ack to
avoid unnecessary string concatenation.

Feel free to use it however you like!

Thanks for the patch ! It is now commited and acknowledged on our team
and contributors page.

Eventually we might consider using larger thread pool than 1 or making
it's size configurable.

Also you might be interested in java wrapper for usrsctp stack[1][2].
I haven't measured it's performance yet, but I expect it might be
faster than PseudoTCP. It can also be run on ice4j sockets.

[1]: https://github.com/jitsi/libjitsi/tree/master/src/org/jitsi/sctp4j
[2]: https://code.google.com/p/sctp-refimpl/

Regards,
Pawel

···

On Tue, Jun 24, 2014 at 3:29 PM, Carl Hasselskog <carl@degoo.com> wrote: