Skip to content

Comments

ARTEMIS-865 Remove e.printStackTrace() from code#896

Closed
treblereel wants to merge 1 commit intoapache:masterfrom
treblereel:JBEAP-7375
Closed

ARTEMIS-865 Remove e.printStackTrace() from code#896
treblereel wants to merge 1 commit intoapache:masterfrom
treblereel:JBEAP-7375

Conversation

@treblereel
Copy link
Contributor

@clebertsuconic
Copy link
Contributor

nice one.. thanks

@treblereel
Copy link
Contributor Author

strange error ... line 803 contains no code :
[INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ artemis-openwire-protocol ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 12 source files to /home/jenkins/jenkins-slave/workspace/ActiveMQ-Artemis-PR-Build/artemis-protocols/artemis-openwire-protocol/target/classes
/home/jenkins/jenkins-slave/workspace/ActiveMQ-Artemis-PR-Build/artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java:803: error: logger has private access in AbstractRemotingConnection
logger.warn("Failed to fire advisory on " + topic, e);

@asfgit asfgit closed this in 5f7f078 Nov 23, 2016
@clebertsuconic
Copy link
Contributor

I fixed it before merging.. it was using the private logger from a super class

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants