Return-Path: Delivered-To: apmail-directory-dev-archive@www.apache.org Received: (qmail 2524 invoked from network); 29 Nov 2005 09:11:56 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 29 Nov 2005 09:11:56 -0000 Received: (qmail 65790 invoked by uid 500); 29 Nov 2005 09:11:55 -0000 Delivered-To: apmail-directory-dev-archive@directory.apache.org Received: (qmail 65751 invoked by uid 500); 29 Nov 2005 09:11:54 -0000 Mailing-List: contact dev-help@directory.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Apache Directory Developers List" Delivered-To: mailing list dev@directory.apache.org Received: (qmail 65740 invoked by uid 99); 29 Nov 2005 09:11:54 -0000 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [192.87.106.226] (HELO ajax.apache.org) (192.87.106.226) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 Nov 2005 01:11:52 -0800 Received: from ajax.apache.org (ajax.apache.org [127.0.0.1]) by ajax.apache.org (Postfix) with ESMTP id 2059DCB for ; Tue, 29 Nov 2005 10:11:31 +0100 (CET) Message-ID: <1985158442.1133255491130.JavaMail.jira@ajax.apache.org> Date: Tue, 29 Nov 2005 10:11:31 +0100 (CET) From: "Niklas Therning (JIRA)" To: dev@directory.apache.org Subject: [jira] Assigned: (DIRMINA-131) Get rid of complexity of the current IoFilterChain. In-Reply-To: <580826219.1133187279898.JavaMail.jira@ajax.apache.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N [ http://issues.apache.org/jira/browse/DIRMINA-131?page=all ] Niklas Therning reassigned DIRMINA-131: --------------------------------------- Assign To: Trustin Lee (was: Niklas Therning) Reassign to Trustin since he's already working on this issue. > Get rid of complexity of the current IoFilterChain. > --------------------------------------------------- > > Key: DIRMINA-131 > URL: http://issues.apache.org/jira/browse/DIRMINA-131 > Project: Directory MINA > Type: Improvement > Versions: 0.8 > Reporter: Trustin Lee > Assignee: Trustin Lee > Fix For: 0.9 > > This issue is the summary of these threads in the Apache Directory Project mailing list: > http://www.nabble.com/-mina-Refactoring-MINA-IoFilterChain-%28Was%3A-IoFilters%3A-DIRMINA-121-122%29-t553121.html > http://www.nabble.com/-mina-IoFilters%3A-DIRMINA-121-122-t548297.html > Currently, IoFilterChains are categorized into two; one is IoSessionManagerFilterChain, and the other is IoSessionFilterChain. IoSessionManagerFilterChain is shared by all sessions managed by the same IoSessionManager, and IoSessionFilterChain is individual for each session. But this design made the internal architecture of MINA filter chain very complex comparing its usefulness. > What about just getting rid of the original IoSessionManagerFilterChain, and provide a simple data structure which just stores a list of filters but still implements IoFilter interface? By doing so, we can simply copy the chain into the IoSessionFilterChain before we start the communication, so the chain implementation gets simplified dramatically. > Besides that, we have another option instead of using a simple data structure that implements IoFilterChain. It is called 'IoFilterChainBuilder'. It is a kind of command pattern which configures the filter chain instead of simply appending the filter list the chain. It provides great flexibility. For example, you can override the settings of per-manager filter chain in your per-port (or per-session) chain builder. > I think it will be best if we can combine these two approach by providing an IoFilterChainBuilder implementation which simply appends the specified filter list to the per-session chain. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira