Made getTransactionManager() protected instead of private

[ljnelson]

This pull request makes the getTransactionManager() method in TransactionContext protected instead of private. In my use case, I have no need of JNDI. It would be nice to be able to override this method so that I can simply use, for example, com.arjuna.ats.jta.common.TransactionManager.transactionManager().

Signed-off-by: Laird Nelson ljnelson@gmail.com

[jbosstm-bot]

⚠️ CI not started.

Author is not the 'narayana' contributor, members of jbosstm can write comment of text to permit PR being run.

[jbosstm-bot]

⚠️ narayana CI not started.

Author is not the 'narayana' contributor, to permit PR being run members of jbosstm can write comment of text: TESTIT

[mmusgrov]

You could use UserTransaction:

UserTransaction ut = com.arjuna.ats.jta.UserTransaction.userTransaction();

And then if you want to manually enlist resources you could do:

   ut.begin();
   TransactionImple tx = TransactionImple.getTransaction();
   tx.enlistResource(...

[tomjenkinson]

Or if you need a reference on the transaction manager itself you could use:

TransactionManager tm = com.arjuna.ats.jta.TransactionManager.transactionManager();

So, I will close this and if there is something specific that can't be done with either of these then please can you open a discussion on our forum and we would be happy to discuss more: https://developer.jboss.org/en/jbosstm/content?filterID=contentstatus[published]~objecttype~objecttype[thread]

[ljnelson]

OK; will start a discussion. For posterity I'm simply trying to get the transactional interceptors to work in CDI SE without JNDI. All I need for this to work is for the transaction manager to be supplied in a DI-friendly way rather than looked up in JNDI.

[jbosstm-bot]

BLACKTIE profile tests failed on Linux (https://ci-master-jenkins-csb-narayana.cloud.paas.psi.redhat.com/job/btny-pulls-narayana/PROFILE=BLACKTIE,jdk=jdk8.latest,label=swarm/626/): Narayana rebase on master failed. Please rebase it manually

[jitendrakmwt]

Getting so many emails from today , already unsubscribed and still getting emails.

[tomjenkinson]

@jitendrakmwt sorry about that

[tomjenkinson]

@jitendrakmwt it should have stopped, we will need to look into it a bit more

[mmusgrov]

@jitendrakmwt Sorry I was setting up a PR poller job on our upshift installation. I temporarily added a build periodically schedule and it started testing all the PRs. Sorry about that. It is disabled now