java.lang.IllegalStateException: await*() about cloudhopper-smpp HOT 4 CLOSED

gabrielhof@bc6b7fa

from cloudhopper-smpp.

Interesting issue. Can you connect to the SMSC outside of this java library? Perhaps there is a firewall in-between that is immediately closing the socket is why I ask. Can you successfully connect to the socket on the other side via telnet?

from cloudhopper-smpp.

I have noticed this when the library is used in the context of a Proxy (i.e. providing simultaneously a server and client interface). Specifically, I ran into this problem when an underlying thread receives a packet say from a server session, and attempts to send the PDU out via a "client" SmppSession. This behaviour stopped when I inserted a thread-pool + queue between the two (i.e. PDU path: Remote Client -> Server Class -> Queue+ThreadPool -> Client Class -> Remote Server). I needed the queue de-coupling so I didn't look too deeply into why this was occurring, but my feeling was that there is a resource sharing issue in either cloudhopper-smpp or netty.

from cloudhopper-smpp.

Yes, your proxy example is true. NIO/netty is always difficult to engineer with since you need to be aware of the context of which thread is calling a method. The methods called upon reception of a PDU (either in a server or client session) are being called with a thread from the thread pool used by NIO/netty. Netty throws an IllegalStateException if that "receiving" thread is also now "sending" a PDU as well. Your solution of adding an intermediate queue + thread will get rid of the exception.

This isn't documented very well in the library -- but one must be pretty careful about handling PDUs as quickly as possible in any method that "receives" a PDU. Sending a PDU from that same thread context will trigger an exception from Netty, but actually any long blocking operation will significantly slow down performance as well. Those "receiving" threads from Netty are used to process incoming data from the socket pool -- and in order to maximize performance overall, its very important they are processed as fast as possible. I'd probably recommend always immediately throwing those PDUs into a queue and processing them with another thread.

from cloudhopper-smpp.