mina-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel Lecharny <elecha...@apache.org>
Subject Re: [3.0] IoSession attachment
Date Mon, 26 Oct 2009 14:11:10 GMT
Julien Vermillard wrote:
> Hi,
>
> I think about removing IoSessionAttributeMap if what we could expose
> if just a Map. As far as I understand it IoSessionAttributeMap is here
> for adding all the utility methods.
>   
The question is : would anyone want to extend the IoSession API by 
adding (useless) methods ?

If you have a look at HttpSession API, it's damn limited :
|java.lang.Object| || 
<http://tomcat.apache.org/tomcat-5.5-doc/servletapi/javax/servlet/http/HttpSession.html#getAttribute%28java.lang.String%29>|*getAttribute

<http://tomcat.apache.org/tomcat-5.5-doc/servletapi/javax/servlet/http/HttpSession.html#getAttribute%28java.lang.String%29>*(java.lang.String
name)| 

|||java.util.Enumeration | 
<http://tomcat.apache.org/tomcat-5.5-doc/servletapi/javax/servlet/http/HttpSession.html#getAttributeNames%28%29>|*getAttributeNames

<http://tomcat.apache.org/tomcat-5.5-doc/servletapi/javax/servlet/http/HttpSession.html#getAttributeNames%28%29>*()|


|*removeAttribute 
<http://tomcat.apache.org/tomcat-5.5-doc/servletapi/javax/servlet/http/HttpSession.html#removeAttribute%28java.lang.String%29>*(java.lang.String
name)| 

|*setAttribute 
<http://tomcat.apache.org/tomcat-5.5-doc/servletapi/javax/servlet/http/HttpSession.html#setAttribute%28java.lang.String,%20java.lang.Object%29>*(java.lang.String
name, 
java.lang.Object value)|

Nobody ever wanted one more method, and nobody ever wanted to know how 
it is handled by the server (as a Set, a Map, a List, whatever).

IMHO, this is what we should do : keep it dead simple. We don't have 
have to expose a Map. Kill the IoSessionAttributeMap interface.

-- 
--
cordialement, regards,
Emmanuel L├ęcharny
www.iktek.com
directory.apache.org



Mime
View raw message