Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add PrivilegedAction blocks to the FileSystemStore #666

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
3 participants
@stuartwdouglas
Copy link
Contributor

commented Jun 5, 2014

!XTS !BLACKTIE !QA_JTA !QA_JTS_JACORB

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 5, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 5, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 5, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 5, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 5, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 6, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 6, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 6, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 6, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 6, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 6, 2014

@tomjenkinson

This comment has been minimized.

Copy link
Contributor

commented Jun 9, 2014

Hi Stuart,

Please can you sign the CLA for JBTM, we always ask regardless of your employer: https://cla.jboss.org/ its (JBoss Transactions).

Also, would be good to reference an issue in the commit message. If you don't want to do that, can you please just add a comment to say why it is being added so when we look in the history its not refactored out ever by overzealous editors :)

After those I will be very happy to merge this - thanks for the contribution!
Tom

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 10, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 10, 2014

Started testing this pull request with BLACKTIE profile on Windows: http://172.17.131.2/job/btny-pulls-narayana-windows2008/687/

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 10, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 10, 2014

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 10, 2014

BLACKTIE profile tests passed on Windows - Job complete http://172.17.131.2/job/btny-pulls-narayana-windows2008/687/

@jbosstm-bot

This comment has been minimized.

Copy link

commented Jun 10, 2014

@stuartwdouglas

This comment has been minimized.

Copy link
Contributor Author

commented Jun 26, 2014

I signed the CLA and updated the commit with the JIRA number.

@tomjenkinson

This comment has been minimized.

Copy link
Contributor

commented Jun 26, 2014

Thanks Stuart, all the tests have passed so I am merging this now. Thanks again for the contribution - Tom

@tomjenkinson

This comment has been minimized.

Copy link
Contributor

commented Jun 26, 2014

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.