You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mina.apache.org by "Yueyu Lin (JIRA)" <ji...@apache.org> on 2007/07/16 07:45:04 UTC

[jira] Commented: (DIRMINA-382) Provide a close() method that doesn't close the connection until all messages are written.

    [ https://issues.apache.org/jira/browse/DIRMINA-382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12512861 ] 

Yueyu Lin commented on DIRMINA-382:
-----------------------------------

I think it will be enough to add a "close(boolean isFlush)".
The old "close()" may just wrap "close(false)".
It will be good to be compatible with the old applications.

> Provide a close() method that doesn't close the connection until all messages are written.
> ------------------------------------------------------------------------------------------
>
>                 Key: DIRMINA-382
>                 URL: https://issues.apache.org/jira/browse/DIRMINA-382
>             Project: MINA
>          Issue Type: New Feature
>          Components: Core
>            Reporter: Trustin Lee
>             Fix For: 2.0.0-M1
>
>
> Currently, IoSession.close() closes the connection immediately no matter how many messages are not written yet.  Calling close() will discard all pending writes and close the connection immediately.  Although we can add a IoFutureListener.CLOSE to the last WriteFuture, it will be more convenient to provide another close method that doesn't close the connection until all message are written.  
> Adding a boolean parameter to the close method will be fine. Of course, original method without a parameter will be retained.
> To implement this, the implementation should satisfy the following condition.
> 1) IoSession.isClosing() must return true after close() is called no matter what boolean parameter is specified.
> 2) The session should be closed after all messages are written out.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.