[2.0-M2] @EJB injection for Jetty doesn't work

[Prasad Kashyap <go...@gmail.com>]

I reviewed the binaries that Matt has put for review on the following
vote thread -

http://www.nabble.com/-VOTE--2.0-M2-Milestone-ready-for-review-and-JACC-Specs-tf3126768.html

The @EJB injection does not work for Jetty. However, the undeploy of
EJBs now work fine.

These fixes were first put into trunk. My initial suspicion is on the
merge.  Maybe the undeploy fix got in and the jetty hack didn't.

My next suspicion is on the fix itself. I hadn't had a chance to try
the @EJB injection hack for jetty on trunk so far due to build
failures. But I hope to try it now.

Neverthless, the release notes are now out of sync.

Didn't want to pollute the vote thread with this discussion.

Cheers
Prasad

Re: [2.0-M2] @EJB injection for Jetty doesn't work

[Prasad Kashyap <go...@gmail.com>]

My initial suspicion turned out to be correct.

The fix in trunk didn't go into M2. Tim McConnell and I verified this
trunk separately. The jetty hack is there in trunk.

The calculator-stateless sample is here for anybody else that wants to try
http://cwiki.apache.org/confluence/download/attachments/35312/calculator-stateless-ear-2.0-SNAPSHOT.ear?version=1

The wiki page is here -
http://cwiki.apache.org/confluence/display/GMOxDOC20/Using+some+of+EJB+3.0+functionalities

Cheers
Prasad

On 1/27/07, Prasad Kashyap <go...@gmail.com> wrote:
> I reviewed the binaries that Matt has put for review on the following
> vote thread -
>
> http://www.nabble.com/-VOTE--2.0-M2-Milestone-ready-for-review-and-JACC-Specs-tf3126768.html
>
> The @EJB injection does not work for Jetty. However, the undeploy of
> EJBs now work fine.
>
> These fixes were first put into trunk. My initial suspicion is on the
> merge.  Maybe the undeploy fix got in and the jetty hack didn't.
>
> My next suspicion is on the fix itself. I hadn't had a chance to try
> the @EJB injection hack for jetty on trunk so far due to build
> failures. But I hope to try it now.
>
> Neverthless, the release notes are now out of sync.
>
> Didn't want to pollute the vote thread with this discussion.
>
> Cheers
> Prasad
>