One more thing.... We really need a way to generically define metadata and a generic way for an interceptor to obtain class/ejb metadata. Currently, if somebody wants to define a custom interceptor, the only way define class/ejb metadata is to modify and recompile JBoss code. Interceptors should be deployable components. The way I have currently designed the AOP framework, this is possible. It may need some tweaking here and there, but I think the design fits.
Bill > -----Original Message----- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] Behalf Of David > Jencks > Sent: Monday, February 24, 2003 10:37 AM > To: [EMAIL PROTECTED] > Subject: RE: [JBoss-dev] org.jboss.aop.MethodMetaData > > > <big snip> > > > I also want to add that the current interface for Metadata, metadata > > chains, > > and how you configure metadata is open for debate. It probably is not a > > complete definition and I'm open for suggestions. I'm hoping that as I > > port > > some of the current interceptors more requirements will be flushed out. > > Persistence will be the big test. > > I don't have any very solid ideas yet, but I think these would be > improvements: > > 1. Single level lookups with Object keys rather than String: Object > getMetadata(Object key). Why force the guy storing the metadata to use a > concealed hashmap with String keys? > > 2. Put the default metadata in the thing that supplies the chain of > interceptors (Advisor?), and always add it first to what the Invocation > gets. (rather than putting the default metadata in the Invocation > directly. > > I have some code with these ideas, I'll try to commit it or send > it to you > later today. > > I think it will take some experimentation to get something that works > really well. > > david > > > > > Bill > > > > > > > > ------------------------------------------------------- > > This sf.net email is sponsored by:ThinkGeek > > Welcome to geek heaven. > > http://thinkgeek.com/sf > > _______________________________________________ > > Jboss-development mailing list > > [EMAIL PROTECTED] > > https://lists.sourceforge.net/lists/listinfo/jboss-development > > > > > > > ------------------------------------------------------- > This sf.net email is sponsored by:ThinkGeek > Welcome to geek heaven. > http://thinkgeek.com/sf > _______________________________________________ > Jboss-development mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/jboss-development ------------------------------------------------------- This sf.net email is sponsored by:ThinkGeek Welcome to geek heaven. http://thinkgeek.com/sf _______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development