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

Dialog is never released #84

Closed
jaimecasero opened this issue Sep 12, 2016 · 6 comments
Closed

Dialog is never released #84

jaimecasero opened this issue Sep 12, 2016 · 6 comments
Assignees
Labels
Milestone

Comments

@jaimecasero
Copy link
Contributor

https://github.com/RestComm/gmlc/blob/master/core/slee/services/sbbs/src/main/java/org/mobicents/gmlc/slee/MobileCoreNetworkInterfaceSbb.java#L330

after receiveng map response we should close dialog, by invoking dialog.release.

This will prevent to receive the dialogtimeout event, and free resources much previouslly

@FerUy FerUy added this to the 1.0.0 milestone Sep 12, 2016
@FerUy FerUy self-assigned this Sep 12, 2016
@FerUy
Copy link
Contributor

FerUy commented Sep 13, 2016

@jaimecasero just added the dialog release. Let's see how it impacts the automated performance test before resolving #83.

@FerUy
Copy link
Contributor

FerUy commented Sep 13, 2016

Consistently with GMLC-Performance-GetLoc/78, this is closed

@FerUy FerUy closed this as completed Sep 13, 2016
@vetss
Copy link
Contributor

vetss commented Sep 16, 2016

@jaimecasero @FerUy

You do not need to add "close() / release()" staff to onAnyTimeInterrogationResponse() because a dialog is closed / released by a peer (a peer is sending TC-END message that just close / release this dialog)

The most probaly reason of "dialogtimeout event" - if you have too high message load then SCTP channel can not pass all load, then messages came to a peer with delays and when this delay increase invoke / dialog timeout margine then you can see many dialog timeout event in logs.
Try to decrease a message load level.

@vetss vetss reopened this Sep 16, 2016
@FerUy
Copy link
Contributor

FerUy commented Sep 16, 2016

@vetss thanks!
You're right, working on it

FerUy pushed a commit that referenced this issue Sep 16, 2016
… an already released dialog by a TC-END message from the HLR. Issue #84
@FerUy
Copy link
Contributor

FerUy commented Sep 16, 2016

Now MAP ATI is working as it expected (TC-BEG for request, TC-END for response) and dialog properly released. Local tests between posterior build 1.0.65 and MAP server stub are also OK as for the attached trace. MAP serverStub log looks fine in this regard as for http://ci.telestax.com/job/GMLC-Performance-GetLoc/85/artifact/results/serverStub.log (i.e. there are no more dialog timeouts, dialogs are released for each MAP ATI request for the whole test).

@FerUy FerUy closed this as completed Sep 16, 2016
@FerUy
Copy link
Contributor

FerUy commented Sep 19, 2016

Just for the record, http://ci.telestax.com/job/GMLC-Performance-GetLoc/90/ confirms this issue was properly closed as for it concern.

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

No branches or pull requests

3 participants