Skip to content

Supervisor: add restart intensity#32

Open
ysbaddaden wants to merge 4 commits into
mainfrom
feature/supervisor-restart-intensity
Open

Supervisor: add restart intensity#32
ysbaddaden wants to merge 4 commits into
mainfrom
feature/supervisor-restart-intensity

Conversation

@ysbaddaden

Copy link
Copy Markdown
Owner

closes #26

@ysbaddaden ysbaddaden added enhancement supervisor Related to Earl::Supervisor labels Mar 10, 2025
@ysbaddaden ysbaddaden self-assigned this Mar 10, 2025
Comment thread test/supervisor_test.cr Outdated
Comment thread src/supervisor.cr Outdated
Comment thread src/supervisor.cr
#
# Wraps an agent to keep additional metadata such as the number of restart
# and when they happened.
class Supervised

@ysbaddaden ysbaddaden Mar 10, 2025

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not very important because it's an internal API, but I'm not fond of the Supervised name. Maybe Metadata?

@ysbaddaden

Copy link
Copy Markdown
Owner Author

Maybe there are better solutions to count restarts over a period than a Deque 🤔

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

Labels

enhancement supervisor Related to Earl::Supervisor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Supervisor: restart intensity

1 participant