Skip to content

ARTEMIS-1798 DEBUG message bad write method arg count - fix#2006

Closed
sknot-rh wants to merge 1 commit intoapache:masterfrom
sknot-rh:E638
Closed

ARTEMIS-1798 DEBUG message bad write method arg count - fix#2006
sknot-rh wants to merge 1 commit intoapache:masterfrom
sknot-rh:E638

Conversation

@sknot-rh
Copy link
Copy Markdown
Contributor

@sknot-rh sknot-rh commented Apr 10, 2018

When the logging level was set up to DEBUG, FluentPropertyBeanIntrospectorWithIgnores threw an exception at org.apache.activemq.artemis.core.protocol.openwire.OpenWireProtocolManager#setUpInactivityParams because it was not really setting up any property. I have renamed this method to 'configureInactivityParams'. That avoids error message in DEBUG output.

public class OpenWireProtocolManager implements ProtocolManager<Interceptor>, ClusterTopologyListener {
static {
// this is not really a property, needs to be ignored
FluentPropertyBeanIntrospectorWithIgnores.addIgnore(OpenWireProtocolManager.class.getName(), "setUpInactivityParams");
Copy link
Copy Markdown
Contributor

@clebertsuconic clebertsuconic Apr 10, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stanlyDoge Instead of that.. what about renaming the method to configureInactivityParams?

You wouldn't need to do anything with Beans.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stanlyDoge told you so ))

@sknot-rh
Copy link
Copy Markdown
Contributor Author

@clebertsuconic Renamed.

@clebertsuconic
Copy link
Copy Markdown
Contributor

nice one.. thanks.. will wait the check and merge it

@asfgit asfgit closed this in 8d4bf95 Apr 10, 2018
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.

3 participants