EsotericSoftware / EsotericSoftware/kryonet

Async client connect() method

Open
#143 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
1.9k
Forks
411
PR merge metrics
No merged PRs in 30d

Description

Currently connect() method trashes the update thread. That makes handling multiple parallel connections troublesome. For example that may be done by separating calls to update() methods in two threads - first one handles the connecting clients and the second one handles send/recv updates. However I'm not sure how correct is it to handle all connect calls in one thread as latency was high when using single thread only (3-4s single thread vs 1-2ms two update threads). Overall it would be great if connect() was async too without all-time-polling connect-only handling thread that is always active in the profiler.

TcpConnection.java:77
` socket.connect(remoteAddress, timeout); // Connect using blocking mode for simplicity.
`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.