ejb2-container-jboss-beans.xml was Re: jboss integration WAS Re: [jboss-dev] Re: Revert 80237 from AS trunk
Ok this is now in trunk.
Galder, I don't know if you want to do further tests?
As part of this, I introduced an
ejb2-container-jboss-beans.xml
Currently this only provides an instance of EJB2's
UserTransaction this is to avoid hardwiring within
the CCM of what UTs exist.
But in future, this would be a better place
to put an "EJB2 service/config bean" with direct
injection of EJB2 dependencies.
Currently a lot of this is configured on the
EJB2xDeployer such that each ejb gets multiple
dependencies and each has to resolve them repeatedly.
Instead, it would be better to just have a single
reference/dependency to the config bean
that gets things directly injected once.
It also gives you a single place to say
something like "stop all ejbs" by stopping that service.
On Wed, 2008-11-05 at 00:09 +0100, Galder Zamarreno wrote:
Fix tidied up. I tested a snapshot of jboss-integration with AS trunk
and previously introduced regressions are not there any more.
Adrian Brock wrote:
> Galder,
>
> I've committed my stuff so it just needs to tidyup your fix
> before we do a release of the integration project.
>
> On Mon, 2008-11-03 at 13:19 +0100, Adrian Brock wrote:
>> On Fri, 2008-10-31 at 16:43 +0100, Galder Zamarreno wrote:
>>> Hi,
>>>
>>> I've just realised that I needed to add something else to
>>> ServerVMClientUserTransaction in jboss-integration to fix this
>>> inefficiency (https://jira.jboss.org/jira/browse/JBAS-5908) compared to
>>> the solution for EAP.
>>>
>>> jboss-integration 5.0.0.GA has been tagged already but I have no problem
>>> in fixing JBAS-5908 for 5.0.1, so can I commit the remaining code in
>>> https://svn.jboss.org/repos/jbossas/projects/integration/trunk/ ? Note
>>> that I don't need to remove anything from the existing (partial) fix, so
>>> no need to retag jboss-integration right now.
>>>
>> Argh! I said not to release it since I might have to do some
>> changes for EJB3 to use UserTransactions correctly.
>>
>> Although I thought I could get away with not making changes
>> in fact I have to. EJB3 can't use some of the necessary api that
>> is in the appserver.
>>
--
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
Adrian Brock
Chief Scientist
JBoss, a division of Red Hat
xxxxxxxxxxxxxxxxxxxxxxxxxxxx