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: Bug
            Reporter: Emmanuel Lecharny
            Priority: Minor


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.
-
You can reply to this email to add a comment to the issue online.

Reply via email to