Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ZOOKEEPER-2209 - ZooKeeper client in C# for 3.4.6 #31

Closed
wants to merge 10 commits into from

Conversation

shayhatsor
Copy link

Inspired by the work of @ewhauser .
I propose a C# Client that supports the current stable version of ZK 3.4.6.
It was built by using static code conversion tools followed by manual editing and C# implementations of java selector and other java constructs.
A great measure was taken to follow the logic of the java version. In fact, the code is almost identical. Thus allowing easy evolution alongside the java version.

Main features:
•fully .NET async, no explicit threads used
•all relevant unit tests have been converted and passing consistently
•recipes are included, will be converted to async soon
•Code is 100% CoreCLR compliant
•NuGet package - ZooKeeperNetEx is already integrated in Microsoft Project Orleans as the only open-source membership provider.

Current Limitations:

  • The recipes code hasn't been converted to task based async yet, so it still uses explicit threads.
  • Support for system properties (currently the defaults are used).
  • SASL support

@shayhatsor shayhatsor changed the title ZooKeeper client in C# for 3.4.6 ZOOKEEPER-2209 - ZooKeeper client in C# for 3.4.6 Jun 10, 2015
@rgs1
Copy link
Member

rgs1 commented Jun 10, 2015

this is awesome @shayhatsor - mind creating a JIRA for this at https://issues.apache.org/jira/browse/ZOOKEEPER? thanks!

@shayhatsor
Copy link
Author

done as soon as i saw the emails about that :)
https://issues.apache.org/jira/browse/ZOOKEEPER-2209

@rgs1
Copy link
Member

rgs1 commented Jun 10, 2015

thanks, will check!

@shayhatsor shayhatsor force-pushed the CSharp branch 2 times, most recently from 838ae57 to ed35835 Compare June 20, 2015 22:50
@shayhatsor shayhatsor force-pushed the CSharp branch 7 times, most recently from aff34a2 to f0cbc8e Compare July 1, 2015 22:30
@shayhatsor shayhatsor force-pushed the CSharp branch 2 times, most recently from ce08ef1 to 84d8418 Compare July 22, 2015 20:31
@shayhatsor shayhatsor force-pushed the CSharp branch 3 times, most recently from ab990ae to 4a6cc24 Compare August 7, 2015 14:50
@shayhatsor shayhatsor force-pushed the CSharp branch 2 times, most recently from 8d8e372 to 0cc3bd9 Compare August 17, 2015 19:35
@shayhatsor shayhatsor closed this Aug 24, 2015
@shayhatsor shayhatsor deleted the CSharp branch August 24, 2015 17:52
@shayhatsor shayhatsor restored the CSharp branch August 26, 2015 20:00
@shayhatsor shayhatsor reopened this Aug 26, 2015
@shayhatsor shayhatsor closed this Oct 31, 2015
@shayhatsor shayhatsor deleted the CSharp branch October 31, 2015 00:20
ztzg pushed a commit to ztzg/zookeeper that referenced this pull request Nov 20, 2019
Bug fix where 2 candidates can become leaders
mgao0 added a commit to mgao0/zookeeper that referenced this pull request Jan 25, 2022
This PR is second commit of zk backup monitoring implementation. This commit integrates BackupStats with BackupManager.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants