Benjamin Reed updated ZOOKEEPER-499:

    Status: Open  (was: Patch Available)

this looks good pat, but when you first get the logger, why are you using the 
package name? if you are going to use the package name shouldn't you get the 
package from the class file?

in the second test, you get the logger using a package to add an appender, but 
remove using the class. couldn't that cause a problem potentially?

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