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.

Sign up for GitHub

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

Jump to bottom

Rewrite how statistic are exposed for consumption #22

Open
tkowalcz opened this issue Jan 22, 2021 · 0 comments
Open

Rewrite how statistic are exposed for consumption #22

tkowalcz opened this issue Jan 22, 2021 · 0 comments
Labels
core Relates to Core component

Comments

@tkowalcz
Copy link
Owner

tkowalcz commented Jan 22, 2021 •
edited
Loading

After implementing current approach to doing metrics I have identified some drawbacks to how these are exposed.

To access the metrics user has to get hold of the appender object (through ((org.apache.logging.log4j.core.LoggerContext) LogManager.getContext(false))) and set MonitoringModule implementation.

Proposal is to add configuration entry that will enable statistics in the appender as well as specify a transient file (possibly in/dev/shm) that will be mmaped and used to dump statistics in real time. We will provide utility classes to decode that file and send the stats via e.g. dropwizard metrics.
@tkowalcz tkowalcz added core Relates to Core component log4j2-appender Relates to Log4j2 Appender component and removed core Relates to Core component labels Jan 22, 2021
@tkowalcz tkowalcz added core Relates to Core component and removed log4j2-appender Relates to Log4j2 Appender component labels Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core Relates to Core component
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tkowalcz