Hi Yes this seems more correct.
You are welcome to log a JIRA and/or provide a fix as a patch / diff / github PR http://camel.apache.org/support On Wed, Jul 29, 2015 at 1:06 PM, imranrazakhan <imranrazak...@gmail.com> wrote: > Hi, > > If we change closeSession() like below > > current: > private void closeSession() { > if (session != null) { > > session.removeSessionStateListener(this.internalSessionStateListener); > try { > Thread.sleep(1000); > session.unbindAndClose(); > } catch (Exception e) { > LOG.warn("Could not close session " + session); > } > > session = null; > > } > > } > > Suggested: > > private void closeSession() { > if (session != null) { > > session.removeSessionStateListener(this.internalSessionStateListener); > try { > Thread.sleep(1000); > session.unbindAndClose(); > session = null; // if we put here then it will retry for > unbind > } catch (Exception e) { > LOG.warn("Could not close session " + session); > } > > session = null; // remove his line > > } > > } > > An other suggetions also welcome > > > > -- > View this message in context: > http://camel.465427.n5.nabble.com/unbind-bug-of-Apache-Camel-SMPP-tp5770008p5770071.html > Sent from the Camel - Users mailing list archive at Nabble.com. -- Claus Ibsen ----------------- http://davsclaus.com @davsclaus Camel in Action 2nd edition: http://www.manning.com/ibsen2
