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
WFLY-10612 Include EJB's IIOP Binding when EJB is deployed logging #13238
Conversation
* A logger with the category {@code org.jboss.as.ejb3.iiop} used for deployment | ||
* log messages | ||
*/ | ||
EjbLogger IIOP_DEPLOYMENT_LOGGER = Logger.getMessageLogger(EjbLogger.class, "org.jboss.as.ejb3.iiop"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should just use DEPLOYMENT_LOGGER.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, Brian. I have changed accordingly.
@chengfang or @tadamski Please review. |
|
||
@LogMessage(level = INFO) | ||
@Message(id = 516, value = "IIOP bindings for session bean named '%s' in deployment unit '%s' are as follows: %s") | ||
void iiopBindings(final String ComponentName, final String ModuleName, String Name); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some minor style issues: either use final for all 3 params or no final.
ComponentName
-> componentName
, the same for other 2 params.
Thanks @RanabirChakraborty ! |
Issue: https://issues.redhat.com/browse/WFLY-10612