Okay, not to crap on everyone... So that method (List.size()) was 20% of the runtime for
whatever reason.
As an experiment, I changed getChildren() to getChildrenMap() in NodeImpl, so I could mix
in NodeSPI. Then I fixed up a few classes here and there. Life seemed good. This method
seemed fixable...
But then... I reran the tests, but they failed. Oops! I fixed some casts in the
LockingInterceptor. Tests hang. Curious, I searched for any additional dependencies on
TreeCacheProxyImpl. Eclipse found like 93 references (many casts) to TreeCacheProxyImpl.
All those would have to be changed for me to go futher...
My immediate reaction was: Fuck. 90 places in the code now depends directly on this class,
I don't know how many are casts.
The only refactoring that makes sense is to fix existing node classes, retrofit TreeCache
with CacheSPI and get rid of this shit. There's just no way I'd run 2.0 seeing
what's in the sausage now.
And I don't see the value of the new API if it's just a ugly retrofit with worse
performance. "-1" as they say...
View the original post :
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3986743#...
Reply to the post :
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...