JBTM-2222 Do not move the GAV for txframework

[tomjenkinson]

!BLACKTIE !QA_JTA !QA_JTS_JACORB !PERF !MAIN

Requires: jbosstm/jboss-as#36

[jbosstm-bot]

Started testing this pull request with XTS profile: http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana/PROFILE=XTS,jdk=jdk7.latest,label=linux/999/

[jbosstm-bot]

XTS profile tests passed - Job complete http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana/PROFILE=XTS,jdk=jdk7.latest,label=linux/999/

[tomjenkinson]

Hi Gytis,

Please can you have a look. The issue was that the maven coordinates for the old txf jar moved. I happen to agree with the forum poster that this is not the best way to deprecate given the maven distribution.

All my tests seem to suggest its OK. Maybe you have an opinion on where each of the tests ended up (i.e. the compensations vs txf folder).

It require a small AS update which I have linked to.

Thanks,
Tom

[gytis]

Hi Tom,

overall I think it looks correct.
Only not sure about .sagas. test. This doesn't test neither compensations nor txframework directly, but it's useful. And it will be gone once we remove txframework completely. So maybe it might be good to keep it in compensations, just in case?

[tomjenkinson]

Sure, good spot. "Compensations" does depend on "txframework" for some of the impl stuff so that will need moving before deleting the txf stuff later on but the .sagas test could have been missed.

Thanks for the review - once it passes I will get it merged

[gytis]

sure, no problem.

[jbosstm-bot]

Started testing this pull request with XTS profile: http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana/PROFILE=XTS,jdk=jdk7.latest,label=linux/1000/

[jbosstm-bot]

XTS profile tests passed - Job complete http://albany.eng.hst.ams2.redhat.com/job/btny-pulls-narayana/PROFILE=XTS,jdk=jdk7.latest,label=linux/1000/