"adrian(a)jboss.org" wrote : "alesj" wrote : WRT:
https://jira.jboss.org/jira/browse/JBSPRING-1
| |
|
| "It does not work" is not a bug report. What's wrong with it?
|
| If you want to send me private e-mails asking me to look at stuff,
| at least give me something to look at. :-)
|
| Either way, I'll usually just ignore this kind of "twaddle" until the
author puts some work into the question to make it answerable.
|
My WRT wasn't really meant for you to go and do detailed analysis. :-)
I thought the "my actual question is really" made this clear.
The private email was about the CL 2.0.4.GA release,
which Jason did, but didn't update the JIRA.
Since you control that space, I thought I asked there (in private email)
as I don't really see that as a dev question. ;-)
The p.s. was just semi-informal. :-)
"adrian(a)jboss.org" wrote :
| You should use an agent.
|
| The Transformer is only there for backwards compatibility with AOP from before the
agent api was available.
|
| Something that is almost certainly no longer relevant except for some old tests in the
testsuite.
|
| The only difference between the Transformer and the ClassFileTransformer
| is that one is inside ClassLoader.defineClass() - i.e. all classloaders
| while the other is only invoked from our ClassLoader.
|
Agent is what we currently use with that Spring's LWT.
But like Jason says, we control the classloading,
and Spring lets you set per app per environment LWT.
They have this for all other app servers, hence I created ours,
which I hope will eventually 'live' in their code.
"adrian(a)jboss.org" wrote :
| If you want to add classloader specific ClassFileTransformers to the
| BaseClassLoaderPolicy api that's ok with me.
|
Already done.
But I'm still using our Translator + adding ClassFileTransformer2Translator bridge.
"adrian(a)jboss.org" wrote :
| I think a better integration point would be to have a Spring aop deployer like
| our aop deployer to analyse (and in your case modify) the classloader
| based on metadata?
Could be done, but I don't see what this brings over their per app server approach.
View the original post :
http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4222671#...
Reply to the post :
http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&a...