Ivan Kelly commented on ZOOKEEPER-789:

The patch attached passes each member of Notification separatedly to 
printNotification. Would it not be simpler to just pass the whole Notification 
object to printNotification. This way if the Notification object changes 
internally, only printNotification needs to be changed. 

> Improve FLE log messages
> ------------------------
>                 Key: ZOOKEEPER-789
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-789
>             Project: Zookeeper
>          Issue Type: Improvement
>    Affects Versions: 3.3.1
>            Reporter: Flavio Paiva Junqueira
>            Assignee: Flavio Paiva Junqueira
>             Fix For: 3.3.2, 3.4.0
>         Attachments: ZOOKEEPER-789.patch, ZOOKEEPER-789.patch
> Notification messages are quite important to determine what is going with 
> leader election. The main idea of this improvement is name the fields we 
> output in notification log messages.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

Reply via email to