-
Notifications
You must be signed in to change notification settings - Fork 10
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
client how use ring object? #7
Comments
The server implementation is a little buggy currently. What needs to happen is after after every operation on the socket a repoll io_uring request has to be made to re-enable polling on the socket (you can see the code here). There's a few edge cases with the current socket handling and I'm pretty sure if some things are broken, especially when using clients with keep alive connections. I've noticed some of the same errors in my testing as well, but haven't had time to look into them more. I've been trying to get a good API for doing batches of operations that work in a single I'm on holiday this weekend, but this looks like a good test case so I'll continue to test with it as I have time. There's also |
@hodgesds I am a R&D engineer of distributed storage. I am very interested in your project. Our project github.com/chubaofs/chubaofs has now entered the cncf foundation. Our project is written in go language. Your iouring library, perfect it and contribute to us? Or join us to make a world-class storage project? |
Chubaofs looks really interesting! I have to check with my employer to double check licensing/contribution policies but it seems like this library could be useful. I've just gotten around to getting an initial version of multiple writer (fan out) support, but the benchmarks seem to show some work still needs to be done. The same goes for the batching with the I think this project has a lot of potential, but there are quite a few things that need to be worked on to get it production ready. I think I will be looking for contributors and also try to work on making sure that users of the project are up to date with what the current state of the project is. |
@hodgesds |
The text was updated successfully, but these errors were encountered: