Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Tomcat Developers List" <dev@tomcat.apache.org>
Received-SPF: pass (athena.apache.org: local policy)
Message-ID: <471A14D0.1040109@hanik.com>
Date: Sat, 20 Oct 2007 08:46:40 -0600
From: Filip Hanik - Dev Lists <devlists@hanik.com>
User-Agent: Thunderbird 2.0.0.6 (Windows/20070728)
MIME-Version: 1.0
To: Tomcat Developers List <dev@tomcat.apache.org>
Subject: Re: [LOBBYING] Add nio connector to configuration validation
References: <4718CB72.5080303@hanik.com>
 <96e4b5230710191311u588d6c4fqa24ed0c432aefa55@mail.gmail.com>
In-Reply-To: <96e4b5230710191311u588d6c4fqa24ed0c432aefa55@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

Costin Manolache wrote:
> -0
>   
this isn't a new proposal, the implementation has already been done. 
this is just completing it.
we'd have to go back to when the implementation was done to start 
changing it, that's not what I'm doing.

Filip
> The setProperty was done way before JMX, and it's quite hacky. I think
> standardising on JMX-like model is better - i.e. if we make changes to
> setProperty and connector configuration, it should bring it closer to JMX.
>
> In  JMX we have a "void setAttribute(...) throws AttributeNotFound", plus a
> method to get supported attribute names.
>
> So instead of returning the boolean - I think it would be better to add a
> method to list supported attributes ( and with some changes to modeler this
> could be reflected in the mbean - right now setProperty is ignored ). It's a
> bit more complicated - but also more valuable.
>
> Costin
> -0
>
> On 10/19/07, Filip Hanik - Dev Lists <devlists@hanik.com> wrote:
>   
>> Ok, sending a little email to lobby for a useful change. To the contrary
>> of comments and vetoes to this change, it doesn't change existing
>> behavior, it simplifies it, and also adds the NIO connector to the
>> validation of server.xml attributes.
>>
>> It'd be nice to get one more vote, so that we can get it in before the
>> tag.
>>
>> From STATUS
>> * Make server.xml parsing warnings more generic, add support for the NIO
>> connector
>> * Connector is no longer an exception case. Any new component can still
>> use setProperty and return a boolean
>> * to accept or reject the property
>>
>> http://people.apache.org/~fhanik/patches/digester-attribute-warnings.patch
>>
>>
>> Filip
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
>> For additional commands, e-mail: dev-help@tomcat.apache.org
>>
>>
>>     
>
>   
> ------------------------------------------------------------------------
>
> No virus found in this incoming message.
> Checked by AVG Free Edition. 
> Version: 7.5.488 / Virus Database: 269.15.3/1081 - Release Date: 10/19/2007 5:41 PM
>   


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org