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

Investigate possibility for batched DB queries in PoolDBESSource #31084

Open
makortel opened this issue Aug 6, 2020 · 4 comments
Open

Investigate possibility for batched DB queries in PoolDBESSource #31084

makortel opened this issue Aug 6, 2020 · 4 comments

Comments

@makortel
Copy link
Contributor

makortel commented Aug 6, 2020

After all EDModules have been migrated to EventSetup consumes (#31061), framework knows exactly which ESProducts are consumed at which stages of the application. This information opens up opportunity to try to query many or all payloads from the conditions DB on one go. Such batching should reduce overheads of the DB queries. We should investigate this after #31061 has been resolved (at least for production workflows).

@makortel
Copy link
Contributor Author

makortel commented Aug 6, 2020

assign core, db

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 6, 2020

New categories assigned: core,db

@Dr15Jones,@smuzaffar,@ggovi,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 6, 2020

A new Issue was created by @makortel Matti Kortelainen.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@makortel
Copy link
Contributor Author

All EDModules run in IBs have been migrated, and esConsumes is now enforced at run time (#35588). Therefore this optimization could be thought more seriously now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants