-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Improve DAO state store persistence and statistics logging [chimp1984] #5935
Conversation
Add thread count to periodic logged statistics Increase the statistic logging interval from 5 minutes to 1 hour
To increase the logging frequency of number of threads to 60 min would defeat the intention to learn about potential issues in cases of lost messages/network issues. It will be less likely that the log would fall close to the period when the issue happens. Maybe we can use a separate periodic log for the number of threads with a rather short period (at least less than 15 min, better shorter). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
Just noticed we're already logging the thread count every 10 minutes:
example
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK based on #5935 (review)
The first commit in this PR (9378fe4) , written by @chimp1984, aims to improve the persistence of DAO state store files.
The motivation was from several user issues (#5929, #5917, #5925, #5934) which cropped up during the upgrade from 1.7.5 to 1.8.0. Each log file provided in the issues showed an error during the persistence of DAO state store files while shutting down v1.7.5 (likely timing related) which then caused a problem of corrupted DAO state files manifested when v1.8.0 started up. The resolution for that is to delete both DAO state files and let them be regenerated by the app at startup.
I have not managed to reproduce the users' issue, but I have tested this PR over several days, both regtest and mainnet. The types of test performed were:
The second commit (0b66fdc) changes the stats logging to include the count of threads and adjust the frequency of logging from 5 minutes to 1 hour. This change of frequency is requested by myself after checking many logs over the past 12 months the connection statistics being repeated every 5 minutes was too frequent, getting in the way of being able to read the log efficiently.
Here is an example of the statistics logging: