[ https://issues.apache.org/jira/browse/ZOOKEEPER-2960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16393875#comment-16393875 ]
ASF GitHub Bot commented on ZOOKEEPER-2960: ------------------------------------------- ijuma opened a new pull request #4678: MINOR: Revert to ZooKeeper 3.4.10 due to ZOOKEEPER-2960 URL: https://github.com/apache/kafka/pull/4678 It's a critical bug that only affects the server, but we don't have an easy way to use 3.4.11 for the zookeeper client only. For reference the upgrade to ZooKeeper 3.4.11 was done via 2652565d429138c58. Testing strategy: relying on existing tests and reverted a change to a test to pass with 3.4.10. ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including upgrade notes) ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > The dataDir and dataLogDir are used opposingly > ---------------------------------------------- > > Key: ZOOKEEPER-2960 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2960 > Project: ZooKeeper > Issue Type: Bug > Components: server > Affects Versions: 3.4.11 > Environment: Not relevant. > Reporter: Dan Milon > Assignee: Andor Molnar > Priority: Critical > Fix For: 3.4.12 > > > _emphasized text_After upgrading from zookeeper 3.4.5, to 3.4.11, without > editing {{zoo.cfg}}, the new version of the server tries to use the > {{dataDir}} as the {{dataLogDir}}, and the {{dataLogDir}} as the {{dataDir}}. > Or at least some parts of the server. > Configuration file has: > {noformat} > $ grep -i data /etc/zookeeper/zoo.cfg > dataLogDir=/var/lib/zookeeper/datalog > dataDir=/var/lib/zookeeper/data > {noformat} > But runtime configuration has: > {noformat} > $ echo conf | nc localhost 2181 | grep -i data > dataDir=/var/lib/zookeeper/datalog/version-2 > dataLogDir=/var/lib/zookeeper/data/version-2 > {noformat} > Also, I got this in the debug logs, so clearly some parts of the server > confuse things. > {noformat} > [PurgeTask:FileTxnSnapLog@79] - Opening datadir:/var/lib/zookeeper/datalog > snapDir:/var/lib/zookeeper/data > [main:FileTxnSnapLog@79] - Opening datadir:/var/lib/zookeeper/data > snapDir:/var/lib/zookeeper/datalog > {noformat} > I tried to look in the code for wrong uses of the directories. I only found > [ZookeeperServer.java|https://github.com/apache/zookeeper/blob/master/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java#L227] > is passing the arguments to {{FileTxnSnapLog}} in the wrong order, but the > code comment says that this is legacy only for tests, so I assume it isn't > the cause for my case. -- This message was sent by Atlassian JIRA (v7.6.3#76005)