Hinterwaeldlers commented on PR #1390: URL: https://github.com/apache/zookeeper/pull/1390#issuecomment-1233986129
Sry for the delay Currently I am not able to reproduce the bug, as connecting within the RO seems to be impossible in 3.7.1 and 3.8 (using the related docker images). In addition the unit test passes with and without the patch. While digging around I've found https://github.com/apache/zookeeper/commit/c47ef905e077184bc5b7f555a3e2dfeb6dc046e1 which could fix the problem, too as no session ID is generated anymore. Related to the RO problem I will create a new bug. Related to this patch: I think adding only the unit test might be the right way to go, assuming the commit above does fix the zxid usage. I will prepare the PR accordingly -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
