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

Logger for NodeMulticastListener doesn't belong to "com.hazelcast" hierarchy #3941

Closed
Mishail opened this issue Oct 29, 2014 · 1 comment
Closed

Comments

@Mishail
Copy link

Mishail commented Oct 29, 2014

Hazelcast 3.3.2
I have following line in my log4j.poperties

log4j.logger.com.hazelcast=WARN, stdout, R

But I still see a lot of INFO messages im my logs

...
13:10:42 [hz._hzInstance_1_dev.MulticastThread] INFO  NodeMulticastListener - [5.5.1.186]:5701 [dev] [3.3.2] Dropped: JoinRequest{packetVersion=2, buildNumber=20141023, address=Address[5.5.1.186]:5701, uuid='97c85107-c4cf-472b-a685-e1b154d79b75', credentials=null, memberCount=1, tryCount=104}
13:10:42 [hz._hzInstance_1_dev.MulticastThread] INFO  NodeMulticastListener - [5.5.1.186]:5701 [dev] [3.3.2] Dropped: JoinRequest{packetVersion=2, buildNumber=20141023, address=Address[5.5.1.186]:5701, uuid='97c85107-c4cf-472b-a685-e1b154d79b75', credentials=null, memberCount=1, tryCount=105}
13:10:42 [hz._hzInstance_1_dev.MulticastThread] INFO  NodeMulticastListener - [5.5.1.186]:5701 [dev] [3.3.2] Dropped: JoinRequest{packetVersion=2, buildNumber=20141023, address=Address[5.5.1.186]:5701, uuid='97c85107-c4cf-472b-a685-e1b154d79b75', credentials=null, memberCount=1, tryCount=106}
...

The reason is NodeMulticastListener doesn't initialize its logger properly:
https://github.com/hazelcast/hazelcast/blob/maintenance-3.x/hazelcast/src/main/java/com/hazelcast/cluster/NodeMulticastListener.java#L39
It should use something like node.getLogger(this.getClass().getName()) instead.

Workaround is to add log4j.logger.NodeMulticastListener=WARN, stdout, R to a config file

@eminn eminn added this to the 3.3.3 milestone Oct 30, 2014
@eminn
Copy link
Contributor

eminn commented Oct 30, 2014

Thanks for the report. Fixed via f01299e

@eminn eminn closed this as completed Oct 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants