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

#1748 Use metrics provided by Micrometer for monitoring executors #1824

Merged
merged 6 commits into from
Apr 17, 2024

Conversation

michal494
Copy link
Contributor

The monitoring of executors is accomplished by a set of metrics implemented by Micrometer instead of manually defining them.

@faderskd
Copy link
Contributor

@michal494 thanks for contribution, nice old code removal.

@faderskd faderskd force-pushed the 1748-add-monitoring-of-executors branch from 5266ec3 to fea4ab2 Compare April 15, 2024 13:52
@faderskd faderskd merged commit 9da6649 into allegro:master Apr 17, 2024
12 checks passed
faderskd pushed a commit that referenced this pull request Apr 19, 2024
)

* #1748 WIP

* #1748 Add monitoring for Zookeeper cache executor service

* #1748 Remove unnecessary method

* #1748 Remove unused methods

* #1748 Remove unused constants

* #1748 Test for monitoring of the executors with Micrometer

---------

Co-authored-by: Michal Ciszewski <michal.ciszewski@allegro.pl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants