[Limitaion] sendRequest function passed to the client is NOT thread-safe – it cannot be called concurrently

[epoberezkin]

That is, can it be used concurrently from multiple threads?

Or should there be something that queues calls to it, like reads/writes to TCP handle?

I've actually already implemented a queueing mechanism, and now just feeling a bit stupid as from looking at source code it does look thread safe, and queueing requests just makes performance worse...

[epoberezkin]

@kazu-yamamoto this is a real issue - this function is not thread safe and cannot be used concurrently.

I currently use queueing that makes sure there are no concurrent calls to this function, but probably it should be thread safe given how API is designed.

[kazu-yamamoto]

I guess this was fixed in a76cdf3.
If you can use sendRequest concurrently with master, please close this issue.

[epoberezkin]

will test that!

[kazu-yamamoto]

Closing.
Please reopen if necessary.