[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mina-dev
Subject:    [jira] [Updated] (DIRMINA-629) The IoServiceStatistics methods are
From:       "Julien Vermillard (JIRA)" <jira () apache ! org>
Date:       2011-08-30 8:38:58
Message-ID: 1042861831.6775.1314693538743.JavaMail.tomcat () hel ! zones ! apache ! org
[Download RAW message or body]


     [ https://issues.apache.org/jira/browse/DIRMINA-629?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

Julien Vermillard updated DIRMINA-629:
--------------------------------------

    Fix Version/s:     (was: 3.0.0-M1)
                   2.0.6

> The IoServiceStatistics methods are called for every new session creation
> -------------------------------------------------------------------------
> 
> Key: DIRMINA-629
> URL: https://issues.apache.org/jira/browse/DIRMINA-629
> Project: MINA
> Issue Type: Improvement
> Affects Versions: 2.0.0-M3
> Reporter: Emmanuel Lecharny
> Priority: Minor
> Fix For: 2.0.6
> 
> 
> When a session is established, some methods of the IoServiceStatistics class are \
> called : protected final void finishSessionInitialization(IoSession session, \
> IoFuture future, IoSessionInitializer sessionInitializer) { if \
> (stats.getLastReadTime() == 0) { \
> ((IoServiceStatistics)stats).setLastReadTime(getActivationTime()); There are two \
>                 problems with this approach :
> - first, many of the members of this classes are not thread safe, leading to some \
>                 random value potentially be put into the stats instance for the \
>                 service
> - second, if we protect those members using some synchronization (or volatile \
> data), it might slow down the connection initialization. The IoServiceStatistics \
> class should be thread safe, and these statistics should not be updated if the user \
> don't want them, ie, it should be optionnal (the Configuration object should take \
> care of this)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


[prev in list] [next in list] [prev in thread] [next in thread]