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

[feature] concurrent safe write function #826

Open
0x-3FFF opened this issue Nov 3, 2022 · 3 comments
Open

[feature] concurrent safe write function #826

0x-3FFF opened this issue Nov 3, 2022 · 3 comments

Comments

@0x-3FFF
Copy link

0x-3FFF commented Nov 3, 2022

currently, the implementation of the write series function is concurrency-unsafe, it is recommended to introduce a concurrency-safe write function

@canelohill
Copy link
Contributor

Concurrent writes are the top support issue for this package.

It's not useful to make the current methods threadsafe given the overall design of the API, but it might be worth doing to reduce the support burden.

@jaitaiwan
Copy link
Member

I've been thinking about this for awhile. Do we support thread-safeness into the future or do we just say "too bad" and ask folks to find a better library. I'm not entirely sure.

@canelohill
Copy link
Contributor

Drive the concurrency-related issues elsewhere by making the individual methods threadsafe. Support thread safety into the future.

Developers of robust applications will want to handle concurrency within their application code. The current API is no hindrance to those developers and may be preferable to other APIs.

I read through the examples in this repository and don't see how additional thread safety in the package will benefit them. Perhaps I need to spend more time looking at them. As an interesting side note, none of the examples use sync.Mutex.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

5 participants