Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reopen: EnquireLink might not work on some conditions ( race condition ) #29

Closed
GoogleCodeExporter opened this issue Oct 9, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

Reopen of issue 13.
http://code.google.com/p/jsmpp/issues/detail?id=13

This issue does NOT require multiple processors.

The PDUReaderWorker thread calls readPDU prior to the main thread setting
the SoTimeout.  Until that read finishes the SoTimeout will not take
affect, so the enquireLink messages won't start until a transaction occurs.

Below is the logs with some added debug statements.

15:29:21.114 INFO  session.SMPPSession$PDUReaderWorker:535 [Starting
PDUReaderWorker with processor degree:3 ...] <Thread-4> 15:29:21.114 DEBUG
session.SMPPSession$PDUReaderWorker:537 [Calling readPDU()] <Thread-4>
15:29:21.196 DEBUG session.SMPPSession$PDUReaderWorker:539 [readPDU() done]
<Thread-4> 15:29:21.196 DEBUG session.SMPPSession$PDUReaderWorker:537
[Calling readPDU()] <Thread-4> 15:29:21.197 DEBUG
session.AbstractSessionContext:82 [Activity notified] <pool-2-thread-1>
15:29:21.197 DEBUG state.SMPPSessionOpen:57 [Bind Response header (31,
80000009, 00000000, 1)] <pool-2-thread-1> 15:29:21.199 DEBUG
session.AbstractSession:258 [bind response received] <main> 15:29:21.200
DEBUG session.SMPPSession$BoundSessionStateListener:654 [setting SoTimeout]
<main> 15:29:21.200 DEBUG session.SMPPSession$BoundSessionStateListener:654
[setting SoTimeout] <main> 

Original issue reported on code.google.com by dchid...@gmail.com on 23 Apr 2009 at 9:06

@GoogleCodeExporter
Copy link
Author

Sorry I just got it.

You and ceruault right.
I'll already fix it. This will applied on the next version.

Original comment by uuda...@gmail.com on 24 Apr 2009 at 6:37

@GoogleCodeExporter
Copy link
Author

Original comment by uuda...@gmail.com on 18 Aug 2009 at 10:48

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant