> -----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? >
Why? Simplicity. But fair nuff. > 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. > Advisor does not work the way you describe it. The Advisor creates an ArrayList of MetaData repositories(the chain) and passes it as a parameter to the Invocation object. The Invocation object is itself a MetadataResolver and is first in the chain. The Invocation object does not have amashed down, merged set of metadata. The Invocation only has a reference to the repositories not a copy of each metadata element. I don't know if this is a good analogy, but I think of it as polymorphic data. > 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. > I agree. I want this chaining, but am open to the implementation. 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
