On 10/24/07, Maarten Bosteels <[EMAIL PROTECTED]> wrote: > On 10/23/07, Trustin Lee <[EMAIL PROTECTED]> wrote: > > > > On 10/23/07, Maarten Bosteels <[EMAIL PROTECTED]> wrote: > > > Hi, > > > > > > Some comments: > > > > > > 1) I understand your worries about performance impact but I think we > > should > > > to a little benchmark before jumping to conclusions. Maybe you already > > did > > > ? > > > > Good idea. I didn't do any benchmark yet. But my primary concern is > > not performance now. It's impedance mismatch. I feel like we are > > working around the problem. We can go as we discussed so far, but > > it's also a lot of fun to explore another possibility. Please don't > > get me wrong. :) > > > > > 2) I was planning to create an Enum of all the keys that > > MdcInjectionFilter > > > supports (remoteAddress, remotePort,...) > > > and add a constructor that takes an EnumSet, that way users can minimize > > the > > > size of the map. > > > > That would be really nice. > > Done, please review.
Great job! I've done some additional tweak: http://svn.apache.org/viewvc?view=rev&revision=588153 Please review. ;) Cheers, Trustin -- what we call human nature is actually human habit -- http://gleamynode.net/ -- PGP Key ID: 0x0255ECA6
