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

Eliminate busy wait on monitor #22

Open
alex-hancock opened this issue Jul 19, 2017 · 0 comments
Open

Eliminate busy wait on monitor #22

alex-hancock opened this issue Jul 19, 2017 · 0 comments

Comments

@alex-hancock
Copy link
Member

The action service monitor (not in this repo, but intertwined with it) wastefully makes URL requests to the volatile Luigi database and to S3, and deals with more information than it needs to.

A possible fix for this problem would be to have the deciders insert the touchfile information into the monitor database, and limit the monitor to updating information from the Consonance job database. This reduces the number of outside calls in the monitor and only results in a minor update in the decider, presumably accompanied by a class to insert into the monitor database without explicitly dealing with database set-up.

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

1 participant