mina-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Emmanuel Lecharny (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DIRMINA-1027) SSLHandler writes corrupt messages under heavy load
Date Fri, 05 Feb 2016 11:10:39 GMT

    [ https://issues.apache.org/jira/browse/DIRMINA-1027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15134018#comment-15134018

Emmanuel Lecharny commented on DIRMINA-1027:

Ah, ok, it's clearer now !

Yeah, I assumed it was not necessary to wrap the {{messageReceived}} part in the protected
loop. It was probably too much an assumption.

Let me check that.

> SSLHandler writes corrupt messages under heavy load
> ---------------------------------------------------
>                 Key: DIRMINA-1027
>                 URL: https://issues.apache.org/jira/browse/DIRMINA-1027
>             Project: MINA
>          Issue Type: Bug
>          Components: SSL
>    Affects Versions: 2.0.11
>            Reporter: Michael Kohlsche
>            Priority: Critical
> I’m facing a critical problem in my project with an MINA-stack including SSL. My Protocol-IoFilterAdapter
receives corrupt messages under heavy load (JMeter-Benchmark with 300 Threads and round about
5-10 MINA-calls per thread). The Problem disappeared without SSL, so I debugged for some days,
and I think the problem occurred because of the changes made for fixing the issue [DIRMINA-1019|https://issues.apache.org/jira/browse/DIRMINA-1019]
([commit|http://git-wip-us.apache.org/repos/asf/mina/commit/9b5c07f9]). I think the problem
is that the SSLHandler is calling messageReceived while also writing data into the next filter,
which is writing inside the messageReceivedEventQueue. With the fix provided inside the race-condition-ticket,
it works like a charm. So I think that the answer of the question "Why the second loop is
also protected by the loop?" is: because otherwise the messageReceivedEventQueue gets corrupted
> (I hope I’m not wrong and anybody can follow my thoughts and maybe bad English)

This message was sent by Atlassian JIRA

View raw message