[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-499?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Patrick Hunt updated ZOOKEEPER-499:
-----------------------------------

    Attachment: ZOOKEEPER-499_br3.2.patch
                ZOOKEEPER-499.patch

updated patch based on ben's comments:

1) using the name of the package is fine, this is actually taking advantage of 
a feature of log4j
http://logging.apache.org/log4j/1.2/manual.html
"Each enabled logging request for a given logger will be forwarded to all the 
appenders in that logger as well as the appenders higher in the hierarchy."

2) I was removing the appender from the wrong logger, I've updated the patch to 
correctly remove.

> electionAlg should default to FLE (3) - regression
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-499
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-499
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: server, tests
>    Affects Versions: 3.2.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Blocker
>             Fix For: 3.2.1, 3.3.0
>
>         Attachments: ZOOKEEPER-499.patch, ZOOKEEPER-499.patch, 
> ZOOKEEPER-499_br3.2.patch, ZOOKEEPER-499_br3.2.patch
>
>
> there's a regression in 3.2 - electionAlg is no longer defaulting to 3 
> (incorrectly defaults to 0)
> also - need to have tests to validate this

-- 
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