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

JBTM-2318 Updated to make some of the messages that commonly appear a… #932

Merged
merged 1 commit into from Nov 13, 2015

Conversation

@tomjenkinson
Copy link
Contributor

commented Oct 26, 2015

https://issues.jboss.org/browse/JBTM-2318

!BLACKTIE !QA_JTA !QA_JTS_JACORB !XTS !PERF !QA_JTS_JDKORB

@tomjenkinson tomjenkinson force-pushed the tomjenkinson:JBTM-2318 branch 2 times, most recently from 7ce04f3 to 52035c0 Oct 26, 2015
@jbosstm-bot

This comment has been minimized.

@jbosstm-bot

This comment has been minimized.

@mmusgrov

This comment has been minimized.

Copy link
Contributor

commented Nov 2, 2015

I have a general question about removing public log messages - do our deprecation rules apply here? If not then the changes look good, otherwise we need to deprecate first.

@tomjenkinson

This comment has been minimized.

Copy link
Contributor Author

commented Nov 13, 2015

Its an astute comment but I really don't think its defined around log message deprecation. I will merge this one therefore. Thanks for the review

@tomjenkinson tomjenkinson force-pushed the tomjenkinson:JBTM-2318 branch from 52035c0 to 2a947c2 Nov 13, 2015
@tomjenkinson tomjenkinson merged commit 2a947c2 into jbosstm:master Nov 13, 2015
@tomjenkinson tomjenkinson deleted the tomjenkinson:JBTM-2318 branch Jun 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.