Process Monitor: Mark process status in DB - #170
Merged
Conversation
iLLiCiTiT
added
the
type: enhancement
iLLiCiTiT
changed the title
MustafaJafar
left a comment
MustafaJafar
left a comment
Member
There was a problem hiding this comment.
My process monitor has a lot of entries. The process monitor window still lags.
Member
Author
I think it needs combination of this and #171 . Created branch |
MustafaJafar
approved these changes
Aug 18, 2026
MustafaJafar
left a comment
MustafaJafar
left a comment
Member
There was a problem hiding this comment.
This works pretty well along #171
# Conflicts: # client/ayon_applications/process.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changelog Description
Store status of a process into DB to avoid re-check of running applications (active/inactive).
Additional review information
The changes were based on #156 but instead of adding a limit it does mark a process in DB with the process status. By default each process is marked as active and it has to be set as inactive on refresh. Once it is marked as inactive it does not check if the process is alive which does enhance the speed of the monitor a lot.
This does change schema of the DB but I would say it is very reliable way how to deal with the slowsness issue, even for future.
WARNING
This does actually change schema of the sqlite database table, it might not be backwards compatible in case this is not merged.
Testing notes: