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

Make DataSvc thread safe #106

Closed
tmadlener opened this issue May 22, 2023 · 3 comments
Closed

Make DataSvc thread safe #106

tmadlener opened this issue May 22, 2023 · 3 comments

Comments

@tmadlener
Copy link
Contributor

As already discussed in #94 eventually we want to move to Gaudi Functional. This requires that our DataHandle / DataSvc works in a thread safe way. #100 introduces Frame based I/O but probably not yet in a thread safe enough way to make it work inside Gaudi Functional.

@tmadlener
Copy link
Contributor Author

@jmcarcell do you want to keep this open until this is done "properly", or can we close this?

@jmcarcell
Copy link
Member

I don't mind either. DataHandles are not used with Gaudi::Functional so the title is a bit wrong on that but it's true that the DataSvc is not thread-safe. And there is a third topic which is moving to Gaudi::Functional which is somehow independent of the other ones (but not complete if the DataSvc is not thread-safe)

@tmadlener tmadlener changed the title Make DataHandle (or DataSvc) work with Gaudi Functional Make DataSvc thread safe Oct 16, 2023
@jmcarcell
Copy link
Member

This is now completed by having a new service that can be used with multithreading IOSvc.

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

No branches or pull requests

2 participants