We can make any required changes. I don't think it was really intentional other than.. nobody extending it.. lets make it final ;)
Once you have a PR / diff whatever I will take a look.. I want to make sure wildfly works.. I think Artemis should have as many Servers as possible integrated. If we can make it with a single RA... even better. (I hope this won't endup with multiple ResourceAdapters imlementations.. one for each server). On Wed, Sep 21, 2016 at 10:33 AM, John D. Ament <john.d.am...@gmail.com> wrote: > Yes, I believe we have a mostly working RA in TomEE now. The main change > was something identified a while ago, the Resource Adapter class has to > have an interface and be non-final. We were able to get that working for > the most part in TomEE, where connection factories are injected from > artemis. We ran into an issue with the MDB listening, but suspect its an > issue on the TomEE side. Do you happen to know if the final declarations > were intentional (and perhaps I'm forgetting something)? > > All in all, it worked pretty well. I have a JIRA to create and Jon F has > the code changes to go in. > > John > > On Wed, Sep 21, 2016 at 7:15 AM Clebert Suconic <clebert.suco...@gmail.com> > wrote: > >> There's the artemis-ra where all the classes are. >> >> That artemis-ra is just for the packaging for the ra.xml >> >> Maybe we should move it somewhere else. >> >> >> >> >> Did you guys get to any code changes required? >> >> On Tue, Sep 20, 2016 at 6:58 PM, John D. Ament <johndam...@apache.org> >> wrote: >> > Hey, >> > >> > Just wrapped up the hacking. Its looking good so far. Just curious, one >> > of the issues that was encountered is that there's no default rar >> generated >> > for Artemis. Should there be? there's an example that ships, maybe it >> > would make sense to include this in the main packages? >> > >> > >> https://github.com/apache/activemq-artemis/tree/master/examples/features/sub-modules/artemis-ra-rar >> > >> > I'm also wondering how other containers may package this up. >> > >> > John >> >> >> >> -- >> Clebert Suconic >> -- Clebert Suconic
